Had ChatGPT fix my BiConsumer cause clearly I'm doing SOMETHING wrong. Lets see if this works.

Author: ArcticRaven

build/classes/java/main/dev/arctic/interactivemenuapi/animation/Animation$1.class

@@ -7,17 +7,32 @@
 import org.bukkit.scheduler.BukkitRunnable;
 import org.bukkit.util.Vector;
 
+/**
+ * Represents an animation that can be applied to elements or divisions in the Interactive Menu API.
+ */
 public class Animation {
     private AnimationType type;
     private double stepper;
     private int duration;
 
+    /**
+     * Constructs an Animation with the specified type, stepper, and tick duration.
+     *
+     * @param type The type of animation.
+     * @param stepper The step value for the animation.
+     * @param tickDuration The duration of the animation in ticks.
+     */
     public Animation(AnimationType type, double stepper, int tickDuration) {
         this.type = type;
         this.stepper = stepper;
         this.duration = tickDuration;
     }
 
+    /**
+     * Applies the animation and returns the result, which includes the vector change and opacity.
+     *
+     * @return The result of the animation, including the vector change and opacity.
+     */
     public AnimationResult apply() {
         Vector vectorChange = new Vector(0, 0, 0);
         int opacity = 100; // Default opacity
@@ -38,6 +53,12 @@ public AnimationResult apply() {
         return new AnimationResult(type, vectorChange, opacity);
     }
 
+    /**
+     * Updates the location of the input division relative to the root menu location.
+     *
+     * @param division The division whose location is to be updated.
+     * @param rootMenuLocation The root location of the menu.
+     */
     public void updateInputDivisionLocation(Division division, Location rootMenuLocation) {
         Plugin plugin = division.getParentMenu().getPlugin();
         new BukkitRunnable() {
@@ -55,57 +76,35 @@ public void run() {
                 if (type != AnimationType.NONE) {
                     AnimationResult result = apply();
                     newLocation.add(result.vectorChange());
-                    setDivisionOpacity(division, result.opacity());
-                }
-
-                division.setCurrentLocation(newLocation);
-
-                for (Element element : division.getElements()) {
-                    element.updateLocation(newLocation);
                 }
 
+                // Update the division's location
+                division.updateLocation(newLocation);
                 ticks++;
             }
-        }.runTaskTimerAsynchronously(plugin, 0, 1);
-    }
-
-    private void setDivisionOpacity(Division division, int opacity) {
-        for (Element element : division.getElements()) {
-            element.setOpacity(opacity);
-        }
+        }.runTaskTimer(plugin, 0, 1);
     }
 
+    /**
+     * Updates the location of the specified element relative to its parent location.
+     *
+     * @param element The element whose location is to be updated.
+     * @param parentLocation The location of the parent.
+     */
     public void updateElementLocation(Element element, Location parentLocation) {
-        Plugin plugin = element.getParentMenu().getPlugin();
-        new BukkitRunnable() {
-            int ticks = 0;
+        Location newLocation = parentLocation.clone().add(element.getOffset());
 
-            @Override
-            public void run() {
-                if (ticks >= duration) {
-                    cancel();
-                    return;
-                }
-
-                Location newLocation = parentLocation.clone().add(element.getOffset());
-
-                if (type != AnimationType.NONE) {
-                    AnimationResult result = apply();
-                    newLocation.add(result.vectorChange());
-                    setElementOpacity(element, result.opacity());
-                }
-
-                element.updateLocation(newLocation);
-
-                ticks++;
-            }
-        }.runTaskTimerAsynchronously(plugin, 0, 1);
-    }
+        if (type != AnimationType.NONE) {
+            AnimationResult result = apply();
+            newLocation.add(result.vectorChange());
+        }
 
-    private void setElementOpacity(Element element, int opacity) {
-        element.setOpacity(opacity);
+        // Update the element's location
+        element.setLocation(newLocation);
     }
 
-    public record AnimationResult(AnimationType type, Vector vectorChange, int opacity) {
-    }
+    /**
+     * Represents the result of an animation, including the type, vector change, and opacity.
+     */
+    public record AnimationResult(AnimationType type, Vector vectorChange, int opacity) {}
 }

build/classes/java/main/dev/arctic/interactivemenuapi/animation/Animation.class

@@ -3,14 +3,14 @@
 import dev.arctic.interactivemenuapi.objects.Division;
 import dev.arctic.interactivemenuapi.objects.Menu;
 import org.bukkit.Location;
-import org.bukkit.util.Vector;
+import org.bukkit.entity.Player;
+import java.util.function.BiConsumer;
 
 /**
  * Interface representing a generic Element in the Interactive Menu API.
  * All specific element types should implement this interface or its sub-interfaces.
  */
 public interface IElement {
-
     /**
      * Updates the location of the element relative to its parent division's location.
      *
@@ -23,16 +23,6 @@ public interface IElement {
      */
     void cleanup();
 
-    /**
-     * Handles interactions with the element. The specific behavior of this method is defined by the
-     * BiConsumer function provided to the element.setOnInteract method.
-     *
-     * @param input The input to the interaction. The type of this parameter is defined by the API user
-     *              when they set the BiConsumer function. It's the responsibility of the API user
-     *              to ensure they're inputting and getting the right type of object.
-     */
-    void onInteract(Object input);
-
     /**
      * Gets the parent menu of this element.
      *
@@ -66,27 +56,27 @@ public interface IElement {
      *
      * @return The current location.
      */
-    Location getCurrentLocation();
+    Location getLocation();
 
     /**
      * Sets the current location of this element.
      *
-     * @param currentLocation The current location.
+     * @param location The current location.
      */
-    void setCurrentLocation(Location currentLocation);
+    void setLocation(Location location);
 
     /**
-     * Gets the offset of this element relative to the parent division's location.
+     * Sets the action to be performed when this element is interacted with.
      *
-     * @return The offset.
+     * @param onInteract BiConsumer that takes a Player and an Object (input).
      */
-    Vector getOffset();
+    void setOnInteract(BiConsumer<Player, Object> onInteract);
 
     /**
-     * Sets the offset of this element relative to the parent division's location.
+     * Executes the action defined by the BiConsumer when this element is interacted with.
      *
-     * @param offset The offset.
+     * @param player The player interacting with the element.
+     * @param input The input object related to the interaction.
      */
-    void setOffset(Vector offset);
+    void onInteract(Player player, Object input);
 }
-

build/classes/java/main/dev/arctic/interactivemenuapi/interfaces/IElement.class

@@ -7,6 +7,7 @@
 import org.bukkit.Color;
 import org.bukkit.Location;
 import org.bukkit.entity.Interaction;
+import org.bukkit.entity.Player;
 import org.bukkit.entity.TextDisplay;
 import org.bukkit.metadata.FixedMetadataValue;
 import org.bukkit.util.Vector;
@@ -15,15 +16,15 @@
 
 @Getter
 @Setter
-public abstract class Element<T> implements IElement {
+public abstract class Element implements IElement {
 
     protected Menu parentMenu;
     protected Division parentDivision;
     protected Location location;
     protected Vector offset;
     protected Interaction interactionEntity;
     protected TextDisplay textDisplayEntity;
-    private BiConsumer<Element<T>, T> onInteract;
+    private BiConsumer<Player, Object> onInteract;
 
     public Element(Menu parentMenu, Division parentDivision, Vector offset) {
         this.parentMenu = parentMenu;
@@ -88,9 +89,19 @@ private Vector getAdjustedOffset(Location anchorLocation, Vector offset, float y
         }
     }
 
-    public void setOnInteract(BiConsumer<Element<T>, T> onInteract) {
+    public void setOnInteract(BiConsumer<Player, Object> onInteract) {
         this.onInteract = onInteract;
     }
 
+    public void onInteract(Player player, Object input) {
+        if (onInteract != null) {
+            onInteract.accept(player, input);
+        }
+    }
+
     public abstract void onInteract(Object input);
+
+    public abstract Location getCurrentLocation();
+
+    public abstract void setCurrentLocation(Location currentLocation);
 }