fix: OOBE completion persistence and UI improvements

- Fix OOBE completion status not persisting after restart
- Add configLoaded flag to prevent saving before config is loaded
- Optimize OOBE responsiveness for small windows (<600px)
- Add animations for tutorial tips switching
- Improve vertical centering and spacing
- Add debug logging for config lifecycle

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: gaoyu06

src/main/java/top/fpsmaster/features/GlobalListener.java

@@ -47,7 +47,7 @@ public void onChatSend(EventSendChatMessage e) {
 
     @Subscribe
     public void onValueChange(EventValueChange e) {
-        if (!FPSMaster.configManager.isLoadingConfig()) {
+        if (FPSMaster.configManager.isConfigLoaded() && !FPSMaster.configManager.isLoadingConfig()) {
             FPSMaster.configManager.saveConfigQuietly("default");
         }
     }

src/main/java/top/fpsmaster/features/impl/interfaces/ClientSettings.java

@@ -30,7 +30,7 @@ public class ClientSettings extends InterfaceModule {
     };
     public static ModeSetting fixedScale = new ModeSetting(
             "FixedScale",
-            5,
+            3,
             () -> fixedScaleEnabled.getValue(),
             "0.5x", "0.75x", "1x", "1.25x", "1.5x", "2x", "2.5x", "3x"
     );

src/main/java/top/fpsmaster/modules/config/ConfigManager.java

@@ -36,13 +36,18 @@ public class ConfigManager {
     private final Gson gson = new GsonBuilder().setPrettyPrinting().create();
     private static final int SCHEMA_VERSION = 1;
     private boolean loadingConfig;
+    private boolean configLoaded;
 
     public Configure configure = new Configure();
 
     public boolean isLoadingConfig() {
         return loadingConfig;
     }
 
+    public boolean isConfigLoaded() {
+        return configLoaded;
+    }
+
     public void saveConfigQuietly(String name) {
         if (loadingConfig) {
             return;
@@ -54,6 +59,7 @@ public void saveConfigQuietly(String name) {
     }
 
     public void saveConfig(String name) throws FileException {
+        System.out.println("[ConfigManager] Saving config: " + name + ", oobeCompleted = " + configure.oobeCompleted);
         JsonObject json = new JsonObject();
         json.addProperty("schemaVersion", SCHEMA_VERSION);
         JsonObject client = new JsonObject();
@@ -186,6 +192,7 @@ public void loadConfig(String name) throws Exception {
                 configure.oobeCompleted = client.has("oobeCompleted")
                         ? client.get("oobeCompleted").getAsBoolean()
                         : FPSMaster.defaultConfigExistedBeforeLoad;
+                System.out.println("[ConfigManager] Loaded oobeCompleted = " + configure.oobeCompleted + " (has field: " + client.has("oobeCompleted") + ", defaultConfigExistedBeforeLoad: " + FPSMaster.defaultConfigExistedBeforeLoad + ")");
                 configure.antiCheatEnabled = !client.has("antiCheatEnabled") || client.get("antiCheatEnabled").getAsBoolean();
                 configure.anonymousDataEnabled = !client.has("anonymousDataEnabled") || client.get("anonymousDataEnabled").getAsBoolean();
                 if (client.has("classicBackgroundColor")) {
@@ -308,6 +315,7 @@ public void loadConfig(String name) throws Exception {
             }
         } finally {
             loadingConfig = false;
+            configLoaded = true;
         }
     }
 

src/main/java/top/fpsmaster/ui/common/TextField.java

@@ -450,7 +450,7 @@ public void drawTextBox(float x, float y, float width, float height) {
             }
 
             boolean flag = j >= 0 && j <= s.length();
-            boolean isFocus = this.isFocused && this.cursorCounter / 6 % 2 == 0 && flag;
+            boolean isFocus = this.isFocused && (System.currentTimeMillis() / 500L) % 2L == 0L && flag;
             float l = this.xPosition + 4;
             float i1 = this.yPosition + (this.height - 8) / 2;
             float j1 = l;

src/main/java/top/fpsmaster/ui/screens/oobe/OobeButton.java

@@ -3,6 +3,7 @@
 import top.fpsmaster.FPSMaster;
 import top.fpsmaster.ui.common.control.UiControl;
 import top.fpsmaster.utils.math.anim.AnimClock;
+import top.fpsmaster.utils.math.anim.AnimMath;
 import top.fpsmaster.utils.math.anim.ColorAnimator;
 import top.fpsmaster.utils.render.draw.Hover;
 import top.fpsmaster.utils.render.draw.Rects;
@@ -22,6 +23,8 @@ public class OobeButton implements UiControl {
     private float height;
     private boolean primary;
     private boolean enabled = true;
+    private float hoverAnim;
+    private float pressAnim;
 
     public OobeButton(String text, boolean primary, Runnable onClick) {
         this.text = text;
@@ -51,15 +54,24 @@ public void render(float x, float y, float width, float height, float mouseX, fl
         this.y = y;
         this.width = width;
         this.height = height;
-        colorAnimator.update(clock.tick());
+        float dt = (float) clock.tick();
+        colorAnimator.update(dt);
         boolean hovered = enabled && Hover.is(x, y, width, height, (int) mouseX, (int) mouseY);
+        hoverAnim = (float) AnimMath.base(hoverAnim, hovered ? 1.0 : 0.0, 0.22);
+        pressAnim = (float) AnimMath.base(pressAnim, 0.0, 0.30);
         Color target = primary
                 ? (hovered ? new Color(118, 131, 252, 242) : new Color(104, 117, 247, enabled ? 228 : 148))
                 : (hovered ? new Color(255, 255, 255, 228) : new Color(255, 255, 255, enabled ? 188 : 145));
         colorAnimator.base(target);
-        Rects.rounded(Math.round(x), Math.round(y), Math.round(width), Math.round(height), 12, colorAnimator.getColor().getRGB());
+        float inset = pressAnim * 1.5f;
+        float drawX = x + inset;
+        float drawY = y + inset;
+        float drawWidth = Math.max(4f, width - inset * 2f);
+        float drawHeight = Math.max(4f, height - inset * 2f);
+        Color fillColor = colorAnimator.getColor();
+        Rects.rounded(Math.round(drawX), Math.round(drawY), Math.round(drawWidth), Math.round(drawHeight), 12, fillColor.getRGB());
         int textColor = primary ? Color.WHITE.getRGB() : new Color(42, 52, 78, enabled ? 255 : 160).getRGB();
-        FPSMaster.fontManager.s14.drawCenteredString(text, x + width / 2f, y + height / 2f - 3.5f, textColor);
+        FPSMaster.fontManager.s16.drawCenteredString(text, x + width / 2f, y + height / 2f - 4.5f + pressAnim * 0.5f, textColor);
     }
 
     @Override
@@ -77,6 +89,7 @@ public void renderInScreen(ScaledGuiScreen screen, float x, float y, float width
     @Override
     public void mouseClicked(float mouseX, float mouseY, int button) {
         if (button == 0 && enabled && Hover.is(x, y, width, height, (int) mouseX, (int) mouseY)) {
+            pressAnim = 1.0f;
             onClick.run();
         }
     }

src/main/java/top/fpsmaster/ui/screens/oobe/OobeDropdown.java

@@ -1,21 +1,30 @@
 package top.fpsmaster.ui.screens.oobe;
 
+import net.minecraft.util.ResourceLocation;
+import org.lwjgl.opengl.GL11;
 import top.fpsmaster.FPSMaster;
 import top.fpsmaster.utils.math.anim.AnimClock;
+import top.fpsmaster.utils.math.anim.AnimMath;
 import top.fpsmaster.utils.render.draw.Hover;
+import top.fpsmaster.utils.render.draw.Images;
 import top.fpsmaster.utils.render.draw.Rects;
 import top.fpsmaster.utils.render.gui.ScaledGuiScreen;
 
 import java.awt.Color;
 
 public class OobeDropdown {
+    private static final ResourceLocation ARROW_ICON = new ResourceLocation("client/gui/settings/icons/arrow.png");
+
     private String label = "";
     private String[] items = new String[0];
     private int selectedIndex;
     private boolean open;
     private boolean enabled = true;
     private float openProgress;
     private final AnimClock clock = new AnimClock();
+    private boolean selectionChanged;
+    private float hoverAnim;
+    private float pressAnim;
 
     public OobeDropdown setLabel(String label) {
         this.label = label;
@@ -47,39 +56,87 @@ public int getSelectedIndex() {
         return selectedIndex;
     }
 
+    public boolean consumeSelectionChanged() {
+        boolean changed = selectionChanged;
+        selectionChanged = false;
+        return changed;
+    }
+
     public void renderInScreen(ScaledGuiScreen screen, float x, float y, float width, float height, int mouseX, int mouseY) {
         float dt = (float) clock.tick();
         float target = open && enabled ? 1f : 0f;
-        openProgress += (target - openProgress) * Math.min(1f, dt * 10f);
+        openProgress += (target - openProgress) * Math.min(1f, dt * 12f);
+        boolean hovered = enabled && Hover.is(x, y, width, height, mouseX, mouseY);
+        hoverAnim = (float) AnimMath.base(hoverAnim, hovered ? 1.0 : 0.0, 0.22);
+        pressAnim = (float) AnimMath.base(pressAnim, 0.0, 0.28);
+
+        Color headColor = enabled
+                ? (hovered ? new Color(247, 249, 255, 252) : new Color(244, 247, 255, 248))
+                : new Color(235, 239, 246, 205);
+        Color labelColor = enabled ? new Color(110, 119, 136, 255) : new Color(146, 152, 164, 200);
+        Color valueColor = enabled ? new Color(27, 35, 48, 255) : new Color(126, 132, 144, 200);
+
+        float inset = pressAnim * 1.3f;
+        float drawX = x + inset;
+        float drawY = y + inset;
+        float drawWidth = Math.max(4f, width - inset * 2f);
+        float drawHeight = Math.max(4f, height - inset * 2f);
 
-        Rects.rounded(Math.round(x), Math.round(y), Math.round(width), Math.round(height), 12, new Color(255, 255, 255, enabled ? 224 : 168).getRGB());
-        FPSMaster.fontManager.s16.drawString(label, x + 12f, y + 11f, new Color(58, 68, 96, enabled ? 255 : 140).getRGB());
+        Rects.rounded(Math.round(drawX), Math.round(drawY), Math.round(drawWidth), Math.round(drawHeight), 14, headColor.getRGB());
+        FPSMaster.fontManager.s16.drawString(label, drawX + 14f, drawY + 10f, labelColor.getRGB());
         String selected = items.length == 0 ? "" : items[selectedIndex];
-        FPSMaster.fontManager.s16.drawString(selected, x + width - 64f, y + 11f, new Color(58, 68, 96, enabled ? 255 : 140).getRGB());
-        FPSMaster.fontManager.s16.drawString(openProgress > 0.5f ? "^" : "v", x + width - 24f, y + 11f, new Color(58, 68, 96, enabled ? 255 : 140).getRGB());
+        float valueX = drawX + drawWidth - 20f - FPSMaster.fontManager.s16.getStringWidth(selected) - 16f;
+        FPSMaster.fontManager.s16.drawString(selected, Math.max(drawX + 74f, valueX), drawY + 10f + pressAnim * 0.5f, valueColor.getRGB());
+        renderArrow(drawX + drawWidth - 18f, drawY + drawHeight / 2f, openProgress, new Color(104, 117, 247, 230));
 
         if (enabled && screen.consumePressInBounds(x, y, width, height, 0) != null) {
+            pressAnim = 1.0f;
             open = !open;
         }
 
         if (openProgress > 0.02f && enabled) {
+            float optionHeight = 24f;
+            float optionGap = 3f;
+            float panelHeight = items.length * optionHeight + Math.max(0, items.length - 1) * optionGap + 10f;
+            float panelY = y + height + 6f;
+            int panelAlpha = Math.max(0, Math.min(255, (int) (openProgress * 255f)));
+
+            Rects.rounded(Math.round(x), Math.round(panelY), Math.round(width), Math.round(panelHeight * openProgress), 10,
+                    new Color(255, 255, 255, Math.min(244, panelAlpha)).getRGB());
+
             for (int i = 0; i < items.length; i++) {
-                float optionY = y + height + 6f + i * 32f * openProgress;
+                float optionY = panelY + 5f + i * (optionHeight + optionGap);
                 int alpha = Math.max(0, Math.min(255, (int) (openProgress * 255f)));
-                Rects.rounded(Math.round(x), Math.round(optionY), Math.round(width), 26, 10,
-                        (i == selectedIndex ? new Color(122, 139, 255, Math.min(230, alpha)) : new Color(255, 255, 255, Math.min(232, alpha))).getRGB());
-                FPSMaster.fontManager.s16.drawString(items[i], x + 12f, optionY + 9f,
-                        (i == selectedIndex ? new Color(255, 255, 255, alpha) : new Color(58, 68, 96, alpha)).getRGB());
-                if (openProgress > 0.95f && screen.consumePressInBounds(x, optionY, width, 28f, 0) != null) {
+                boolean optionHovered = Hover.is(x + 4f, optionY, width - 8f, optionHeight, mouseX, mouseY);
+                Color optionColor = i == selectedIndex
+                        ? new Color(104, 117, 247, Math.min(228, alpha))
+                        : (optionHovered ? new Color(240, 244, 255, Math.min(236, alpha)) : new Color(255, 255, 255, 0));
+                if (optionColor.getAlpha() > 0) {
+                    Rects.rounded(Math.round(x + 4f), Math.round(optionY), Math.round(width - 8f), Math.round(optionHeight), 10, optionColor.getRGB());
+                }
+                FPSMaster.fontManager.s16.drawString(items[i], x + 12f, optionY + 6f,
+                        (i == selectedIndex ? new Color(255, 255, 255, alpha) : new Color(78, 89, 108, alpha)).getRGB());
+                if (openProgress > 0.95f && screen.consumePressInBounds(x, optionY, width, optionHeight, 0) != null) {
                     selectedIndex = i;
+                    selectionChanged = true;
+                    pressAnim = 1.0f;
                     open = false;
                 }
             }
 
             ScaledGuiScreen.PointerEvent outside = screen.peekAnyPress();
-            if (outside != null && openProgress > 0.95f && !Hover.is(x, y, width, height + items.length * 32f + 12f, outside.x, outside.y)) {
+            if (outside != null && openProgress > 0.95f && !Hover.is(x, y, width, height + panelHeight + 4f, outside.x, outside.y)) {
                 open = false;
             }
         }
     }
+
+    private void renderArrow(float centerX, float centerY, float progress, Color color) {
+        GL11.glPushMatrix();
+        GL11.glTranslatef(centerX, centerY, 0f);
+        GL11.glRotatef(progress * 180f, 0f, 0f, 1f);
+        GL11.glTranslatef(-centerX, -centerY, 0f);
+        Images.draw(ARROW_ICON, centerX - 4f, centerY - 4f, 8f, 8f, color);
+        GL11.glPopMatrix();
+    }
 }