1.4.3

Cache search query results for huge performance gains
Allow some `StringControllerElement` methods to be inheritable

Author: isXander

build.gradle.kts

@@ -16,7 +16,7 @@ plugins {
 val ciRun = System.getenv().containsKey("GITHUB_ACTIONS")
 
 group = "dev.isxander"
-version = "1.4.2"
+version = "1.4.3"
 
 if (ciRun)
     version = "$version-SNAPSHOT"

changelogs/1.4.3.md

@@ -0,0 +1,2 @@
+- Cache search query results for huge performance gains
+- Allow some `StringControllerElement` methods to be inheritable

src/main/java/dev/isxander/yacl/gui/OptionListWidget.java

@@ -26,6 +26,8 @@ public class OptionListWidget extends ElementListWidget<OptionListWidget.Entry>
     private final YACLScreen yaclScreen;
     private boolean singleCategory = false;
 
+    private ImmutableList<Entry> viewableChildren;
+
     public OptionListWidget(YACLScreen screen, MinecraftClient client, int width, int height) {
         super(client, width / 3 * 2, height, 0, height, 22);
         this.yaclScreen = screen;
@@ -71,6 +73,7 @@ public void refreshOptions() {
             }
         }
 
+        recacheViewableChildren();
         setScrollAmount(0);
     }
 
@@ -203,9 +206,13 @@ protected void renderBackground(MatrixStack matrices) {
             fill(matrices, left, top, right, bottom, 0x6B000000);
     }
 
+    public void recacheViewableChildren() {
+        this.viewableChildren = ImmutableList.copyOf(super.children().stream().filter(Entry::isViewable).toList());
+    }
+
     @Override
     public List<Entry> children() {
-        return super.children().stream().filter(Entry::isViewable).toList();
+        return viewableChildren;
     }
 
     public abstract class Entry extends ElementListWidget.Entry<Entry> {
@@ -322,6 +329,7 @@ private GroupSeparatorEntry(OptionGroup group, Screen screen) {
             this.groupExpanded = !group.collapsed();
             this.expandMinimizeButton = new LowProfileButtonWidget(0, 0, 20, 20, Text.empty(), btn -> {
                 setExpanded(!isExpanded());
+                recacheViewableChildren();
             });
             updateExpandMinimizeText();
         }

src/main/java/dev/isxander/yacl/gui/SearchFieldWidget.java

@@ -31,19 +31,15 @@ public void renderButton(MatrixStack matrices, int mouseX, int mouseY, float del
     @Override
     public void write(String text) {
         update();
-
         super.write(text);
-
-        isEmpty = getText().isEmpty();
+        postUpdate();
     }
 
     @Override
     public void eraseCharacters(int characterOffset) {
         update();
-
         super.eraseCharacters(characterOffset);
-
-        isEmpty = getText().isEmpty();
+        postUpdate();
     }
 
     private void update() {
@@ -56,6 +52,11 @@ private void update() {
         }
     }
 
+    private void postUpdate() {
+        isEmpty = getText().isEmpty();
+        yaclScreen.optionList.recacheViewableChildren();
+    }
+
     public boolean isEmpty() {
         return isEmpty;
     }

src/main/java/dev/isxander/yacl/gui/controllers/slider/SliderControllerElement.java

@@ -119,19 +119,22 @@ public boolean isMouseOver(double mouseX, double mouseY) {
         return super.isMouseOver(mouseX, mouseY) || mouseDown;
     }
 
-    private void setValueFromMouse(double mouseX) {
+    protected void setValueFromMouse(double mouseX) {
         double value = (mouseX - sliderBounds.x()) / sliderBounds.width() * control.range();
-        double roundedValue = MathHelper.clamp(min + (interval * Math.round(value / interval)), min, max); // extremely imprecise, requires clamping
-        control.setPendingValue(roundedValue);
+        control.setPendingValue(roundToInterval(value));
         calculateInterpolation();
     }
 
+    protected double roundToInterval(double value) {
+        return MathHelper.clamp(min + (interval * Math.round(value / interval)), min, max); // extremely imprecise, requires clamping
+    }
+
     @Override
     protected int getHoveredControlWidth() {
         return sliderBounds.width() + getUnhoveredControlWidth() + 6 + getThumbWidth() / 2;
     }
 
-    private void calculateInterpolation() {
+    protected void calculateInterpolation() {
         interpolation = (float) ((control.pendingValue() - control.min()) * 1 / control.range());
     }
 
@@ -141,11 +144,11 @@ public void setDimension(Dimension<Integer> dim) {
         sliderBounds = Dimension.ofInt(dim.xLimit() - getXPadding() - getThumbWidth() / 2 - dim.width() / 3, dim.centerY() - 5, dim.width() / 3, 10);
     }
 
-    private int getThumbX() {
+    protected int getThumbX() {
         return (int) (sliderBounds.x() + sliderBounds.width() * interpolation);
     }
 
-    private int getThumbWidth() {
+    protected int getThumbWidth() {
         return 4;
     }