fix button in open link gui not being centered (#428)

GTNewHorizons · Oct 20, 2024 · 7c0774b · 7c0774b
1 parent 7b12b54
commit 7c0774b
Show file tree

Hide file tree

Showing 3 changed files with 56 additions and 0 deletions.
diff --git a/src/main/java/com/mitchej123/hodgepodge/config/FixesConfig.java b/src/main/java/com/mitchej123/hodgepodge/config/FixesConfig.java
@@ -358,6 +358,11 @@ public class FixesConfig {
                             // settings
     public static int limitRecursiveBlockUpdateDepth;
 
+    @Config.Comment("Fix the buttons not being centered in the GuiConfirmOpenLink")
+    @Config.DefaultBoolean(true)
+    @Config.RequiresMcRestart
+    public static boolean fixButtonsGuiConfirmOpenLink;
+
     @Config.Comment("Fix an array out of bounds caused by the GameSettings getKeyDisplayString method")
     @Config.DefaultBoolean(true)
     @Config.RequiresMcRestart

diff --git a/src/main/java/com/mitchej123/hodgepodge/mixins/Mixins.java b/src/main/java/com/mitchej123/hodgepodge/mixins/Mixins.java
@@ -431,6 +431,11 @@ public enum Mixins {
                     .addMixinClasses("minecraft.MixinWorldServer_LimitUpdateRecursion")
                     .setApplyIf(() -> FixesConfig.limitRecursiveBlockUpdateDepth >= 0)),
 
+    FIX_BUTTON_POS_GUIOPENLINK(new Builder("Fix the buttons not being centered in the GuiConfirmOpenLink")
+            .setPhase(Phase.EARLY).setSide(Side.CLIENT).addTargetedMod(TargetedMod.VANILLA)
+            .addMixinClasses("minecraft.MixinGuiConfirmOpenLink")
+            .setApplyIf(() -> FixesConfig.fixButtonsGuiConfirmOpenLink)),
+
     FIX_CHAT_OPEN_LINK(new Builder("Fix the vanilla method to open chat links not working for every OS")
             .setPhase(Phase.EARLY).setSide(Side.CLIENT).addTargetedMod(TargetedMod.VANILLA)
             .addMixinClasses("minecraft.MixinGuiChat_OpenLinks").setApplyIf(() -> FixesConfig.fixChatOpenLink)),

diff --git a/src/main/java/com/mitchej123/hodgepodge/mixins/early/minecraft/MixinGuiConfirmOpenLink.java b/src/main/java/com/mitchej123/hodgepodge/mixins/early/minecraft/MixinGuiConfirmOpenLink.java
@@ -0,0 +1,46 @@
+package com.mitchej123.hodgepodge.mixins.early.minecraft;
+
+import net.minecraft.client.gui.GuiConfirmOpenLink;
+import net.minecraft.client.gui.GuiScreen;
+
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.injection.At;
+import org.spongepowered.asm.mixin.injection.ModifyArg;
+
+@Mixin(GuiConfirmOpenLink.class)
+public class MixinGuiConfirmOpenLink extends GuiScreen {
+
+    @ModifyArg(
+            method = "initGui",
+            at = @At(
+                    value = "INVOKE",
+                    target = "Lnet/minecraft/client/gui/GuiButton;<init>(IIIIILjava/lang/String;)V",
+                    ordinal = 0),
+            index = 1)
+    private int hodgepodge$fixButton0(int xPos) {
+        return this.width / 2 - 50 - 105;
+    }
+
+    @ModifyArg(
+            method = "initGui",
+            at = @At(
+                    value = "INVOKE",
+                    target = "Lnet/minecraft/client/gui/GuiButton;<init>(IIIIILjava/lang/String;)V",
+                    ordinal = 1),
+            index = 1)
+    private int hodgepodge$fixButton1(int xPos) {
+        return this.width / 2 - 50;
+    }
+
+    @ModifyArg(
+            method = "initGui",
+            at = @At(
+                    value = "INVOKE",
+                    target = "Lnet/minecraft/client/gui/GuiButton;<init>(IIIIILjava/lang/String;)V",
+                    ordinal = 2),
+            index = 1)
+    private int hodgepodge$fixButton2(int xPos) {
+        return this.width / 2 - 50 + 105;
+    }
+
+}