fix category buttons sometimes not being registered as clicked

3 files changed, 109 insertions, 20 deletions

diff --git a/src/client/java/dev/isxander/yacl/impl/ConfigCategoryImpl.java b/src/client/java/dev/isxander/yacl/impl/ConfigCategoryImpl.java
index b3da814..5f9bdd0 100644
--- a/src/client/java/dev/isxander/yacl/impl/ConfigCategoryImpl.java
+++ b/src/client/java/dev/isxander/yacl/impl/ConfigCategoryImpl.java
@@ -17,9 +17,34 @@ import java.util.Collection;
 import java.util.List;
 
 @ApiStatus.Internal
-public record ConfigCategoryImpl(Text name, ImmutableList<OptionGroup> groups, Text tooltip) implements ConfigCategory {
+public class ConfigCategoryImpl implements ConfigCategory {
+    private final Text name;
+    private final ImmutableList<OptionGroup> groups;
+    private final Text tooltip;
+
+    public ConfigCategoryImpl(Text name, ImmutableList<OptionGroup> groups, Text tooltip) {
+        this.name = name;
+        this.groups = groups;
+        this.tooltip = tooltip;
+    }
+
+    @Override
+    public @NotNull Text name() {
+        return name;
+    }
+
+    @Override
+    public @NotNull ImmutableList<OptionGroup> groups() {
+        return groups;
+    }
+
+    @Override
+    public @NotNull Text tooltip() {
+        return tooltip;
+    }
+
     @ApiStatus.Internal
-    public static final class BuilderImpl implements ConfigCategory.Builder {
+    public static final class BuilderImpl implements Builder {
         private Text name;
 
         private final List<Option<?>> rootOptions = new ArrayList<>();
@@ -28,7 +53,7 @@ public record ConfigCategoryImpl(Text name, ImmutableList<OptionGroup> groups, T
         private final List<Text> tooltipLines = new ArrayList<>();
 
         @Override
-        public ConfigCategory.Builder name(@NotNull Text name) {
+        public Builder name(@NotNull Text name) {
             Validate.notNull(name, "`name` cannot be null");
 
             this.name = name;
@@ -36,7 +61,7 @@ public record ConfigCategoryImpl(Text name, ImmutableList<OptionGroup> groups, T
         }
 
         @Override
-        public ConfigCategory.Builder option(@NotNull Option<?> option) {
+        public Builder option(@NotNull Option<?> option) {
             Validate.notNull(option, "`option` must not be null");
 
             if (option instanceof ListOption<?> listOption) {
@@ -49,7 +74,7 @@ public record ConfigCategoryImpl(Text name, ImmutableList<OptionGroup> groups, T
         }
 
         @Override
-        public ConfigCategory.Builder options(@NotNull Collection<Option<?>> options) {
+        public Builder options(@NotNull Collection<Option<?>> options) {
             Validate.notNull(options, "`options` must not be null");
 
             if (options.stream().anyMatch(ListOption.class::isInstance))
@@ -60,7 +85,7 @@ public record ConfigCategoryImpl(Text name, ImmutableList<OptionGroup> groups, T
         }
 
         @Override
-        public ConfigCategory.Builder group(@NotNull OptionGroup group) {
+        public Builder group(@NotNull OptionGroup group) {
             Validate.notNull(group, "`group` must not be null");
 
             this.groups.add(group);
@@ -68,7 +93,7 @@ public record ConfigCategoryImpl(Text name, ImmutableList<OptionGroup> groups, T
         }
 
         @Override
-        public ConfigCategory.Builder groups(@NotNull Collection<OptionGroup> groups) {
+        public Builder groups(@NotNull Collection<OptionGroup> groups) {
             Validate.notEmpty(groups, "`groups` must not be empty");
 
             this.groups.addAll(groups);
@@ -76,7 +101,7 @@ public record ConfigCategoryImpl(Text name, ImmutableList<OptionGroup> groups, T
         }
 
         @Override
-        public ConfigCategory.Builder tooltip(@NotNull Text... tooltips) {
+        public Builder tooltip(@NotNull Text... tooltips) {
             Validate.notEmpty(tooltips, "`tooltips` cannot be empty");
 
             tooltipLines.addAll(List.of(tooltips));
diff --git a/src/client/java/dev/isxander/yacl/impl/OptionGroupImpl.java b/src/client/java/dev/isxander/yacl/impl/OptionGroupImpl.java
index 89a2adf..ba96941 100644
--- a/src/client/java/dev/isxander/yacl/impl/OptionGroupImpl.java
+++ b/src/client/java/dev/isxander/yacl/impl/OptionGroupImpl.java
@@ -15,16 +15,55 @@ import java.util.Collection;
 import java.util.List;
 
 @ApiStatus.Internal
-public record OptionGroupImpl(@NotNull Text name, @NotNull Text tooltip, ImmutableList<? extends Option<?>> options, boolean collapsed, boolean isRoot) implements OptionGroup {
+public class OptionGroupImpl implements OptionGroup {
+    private final @NotNull Text name;
+    private final @NotNull Text tooltip;
+    private final ImmutableList<? extends Option<?>> options;
+    private final boolean collapsed;
+    private final boolean isRoot;
+
+    public OptionGroupImpl(@NotNull Text name, @NotNull Text tooltip, ImmutableList<? extends Option<?>> options, boolean collapsed, boolean isRoot) {
+        this.name = name;
+        this.tooltip = tooltip;
+        this.options = options;
+        this.collapsed = collapsed;
+        this.isRoot = isRoot;
+    }
+
+    @Override
+    public @NotNull Text name() {
+        return name;
+    }
+
+    @Override
+    public @NotNull Text tooltip() {
+        return tooltip;
+    }
+
+    @Override
+    public @NotNull ImmutableList<? extends Option<?>> options() {
+        return options;
+    }
+
+    @Override
+    public boolean collapsed() {
+        return collapsed;
+    }
+
+    @Override
+    public boolean isRoot() {
+        return isRoot;
+    }
+
     @ApiStatus.Internal
-    public static final class BuilderImpl implements OptionGroup.Builder {
+    public static final class BuilderImpl implements Builder {
         private Text name = Text.empty();
         private final List<Text> tooltipLines = new ArrayList<>();
         private final List<Option<?>> options = new ArrayList<>();
         private boolean collapsed = false;
 
         @Override
-        public OptionGroup.Builder name(@NotNull Text name) {
+        public Builder name(@NotNull Text name) {
             Validate.notNull(name, "`name` must not be null");
 
             this.name = name;
@@ -32,7 +71,7 @@ public record OptionGroupImpl(@NotNull Text name, @NotNull Text tooltip, Immutab
         }
 
         @Override
-        public OptionGroup.Builder tooltip(@NotNull Text... tooltips) {
+        public Builder tooltip(@NotNull Text... tooltips) {
             Validate.notEmpty(tooltips, "`tooltips` cannot be empty");
 
             tooltipLines.addAll(List.of(tooltips));
@@ -40,7 +79,7 @@ public record OptionGroupImpl(@NotNull Text name, @NotNull Text tooltip, Immutab
         }
 
         @Override
-        public OptionGroup.Builder option(@NotNull Option<?> option) {
+        public Builder option(@NotNull Option<?> option) {
             Validate.notNull(option, "`option` must not be null");
 
             if (option instanceof ListOption<?>)
@@ -51,7 +90,7 @@ public record OptionGroupImpl(@NotNull Text name, @NotNull Text tooltip, Immutab
         }
 
         @Override
-        public OptionGroup.Builder options(@NotNull Collection<? extends Option<?>> options) {
+        public Builder options(@NotNull Collection<? extends Option<?>> options) {
             Validate.notEmpty(options, "`options` must not be empty");
 
             if (options.stream().anyMatch(ListOption.class::isInstance))
@@ -62,7 +101,7 @@ public record OptionGroupImpl(@NotNull Text name, @NotNull Text tooltip, Immutab
         }
 
         @Override
-        public OptionGroup.Builder collapsed(boolean collapsible) {
+        public Builder collapsed(boolean collapsible) {
             this.collapsed = collapsible;
             return this;
         }
diff --git a/src/client/java/dev/isxander/yacl/impl/PlaceholderCategoryImpl.java b/src/client/java/dev/isxander/yacl/impl/PlaceholderCategoryImpl.java
index 28e5886..4ea623b 100644
--- a/src/client/java/dev/isxander/yacl/impl/PlaceholderCategoryImpl.java
+++ b/src/client/java/dev/isxander/yacl/impl/PlaceholderCategoryImpl.java
@@ -18,14 +18,39 @@ import java.util.List;
 import java.util.function.BiFunction;
 
 @ApiStatus.Internal
-public record PlaceholderCategoryImpl(Text name, BiFunction<MinecraftClient, YACLScreen, Screen> screen, Text tooltip) implements PlaceholderCategory {
+public final class PlaceholderCategoryImpl implements PlaceholderCategory {
+    private final Text name;
+    private final BiFunction<MinecraftClient, YACLScreen, Screen> screen;
+    private final Text tooltip;
+
+    public PlaceholderCategoryImpl(Text name, BiFunction<MinecraftClient, YACLScreen, Screen> screen, Text tooltip) {
+        this.name = name;
+        this.screen = screen;
+        this.tooltip = tooltip;
+    }
+
     @Override
     public @NotNull ImmutableList<OptionGroup> groups() {
         return ImmutableList.of();
     }
 
+    @Override
+    public @NotNull Text name() {
+        return name;
+    }
+
+    @Override
+    public BiFunction<MinecraftClient, YACLScreen, Screen> screen() {
+        return screen;
+    }
+
+    @Override
+    public @NotNull Text tooltip() {
+        return tooltip;
+    }
+
     @ApiStatus.Internal
-    public static final class BuilderImpl implements PlaceholderCategory.Builder {
+    public static final class BuilderImpl implements Builder {
         private Text name;
 
         private final List<Text> tooltipLines = new ArrayList<>();
@@ -33,7 +58,7 @@ public record PlaceholderCategoryImpl(Text name, BiFunction<MinecraftClient, YAC
         private BiFunction<MinecraftClient, YACLScreen, Screen> screenFunction;
 
         @Override
-        public PlaceholderCategory.Builder name(@NotNull Text name) {
+        public Builder name(@NotNull Text name) {
             Validate.notNull(name, "`name` cannot be null");
 
             this.name = name;
@@ -41,7 +66,7 @@ public record PlaceholderCategoryImpl(Text name, BiFunction<MinecraftClient, YAC
         }
 
         @Override
-        public PlaceholderCategory.Builder tooltip(@NotNull Text... tooltips) {
+        public Builder tooltip(@NotNull Text... tooltips) {
             Validate.notEmpty(tooltips, "`tooltips` cannot be empty");
 
             tooltipLines.addAll(List.of(tooltips));
@@ -49,7 +74,7 @@ public record PlaceholderCategoryImpl(Text name, BiFunction<MinecraftClient, YAC
         }
 
         @Override
-        public PlaceholderCategory.Builder screen(@NotNull BiFunction<MinecraftClient, YACLScreen, Screen> screenFunction) {
+        public Builder screen(@NotNull BiFunction<MinecraftClient, YACLScreen, Screen> screenFunction) {
             Validate.notNull(screenFunction, "`screenFunction` cannot be null");
 
             this.screenFunction = screenFunction;