-
Notifications
You must be signed in to change notification settings - Fork 191
Update mod to Minecraft 1.21.9 #903
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
This port should be solid for Release time. |
NotRyken
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found a couple of issues using this while testing my own mods.
src/main/java/com/terraformersmc/modmenu/gui/widget/ModListWidget.java
Outdated
Show resolved
Hide resolved
src/main/java/com/terraformersmc/modmenu/gui/widget/entries/ChildEntry.java
Outdated
Show resolved
Hide resolved
|
See if the changes satisfies the requests. |
NotRyken
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Appears to work correctly.
src/main/java/com/terraformersmc/modmenu/gui/widget/entries/ChildEntry.java
Outdated
Show resolved
Hide resolved
src/main/java/com/terraformersmc/modmenu/gui/widget/entries/ParentEntry.java
Outdated
Show resolved
Hide resolved
src/main/java/com/terraformersmc/modmenu/gui/widget/entries/ParentEntry.java
Outdated
Show resolved
Hide resolved
src/main/java/com/terraformersmc/modmenu/gui/widget/entries/ParentEntry.java
Outdated
Show resolved
Hide resolved
src/main/java/com/terraformersmc/modmenu/gui/widget/ModListWidget.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The 'Toggle Filter Options' and 'Configure...' buttons do not change the cursor shape unlike other buttons. Links, unlike those generated by commands such as /tellraw @s {click_event: {action: open_url, url: "https://example.org"}, text: Example}, also do not change the cursor shape.
In addition, when clicking the 'Website' or 'Issues' buttons and returning to the mods screen, the description of the currently selected mod is not restored.
1.21.9-pre1 Slight cleanup Bump PAPI, fix Icon Alignment by 2 px Bump PRE2 Bump PRE3 Bump PRE4 Bump RC.1, address comments. Bump PAPI Fix formatting Update src/main/java/com/terraformersmc/modmenu/gui/widget/entries/ChildEntry.java Update src/main/java/com/terraformersmc/modmenu/gui/widget/entries/ParentEntry.java Update src/main/resources/fabric.mod.json Update src/main/java/com/terraformersmc/modmenu/gui/widget/entries/ParentEntry.java Update src/main/java/com/terraformersmc/modmenu/gui/widget/ModListWidget.java Co-Authored-By: haykam821 <[email protected]>
6e56a87 to
3698ce6
Compare
|
@haykam821 -- See the changes. There seems to have been a bug with how the DescriptionList was getting updated; by doing a simple |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
… the right side as well.
haykam821
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The layout remains misaligned compared to the previous Minecraft 1.21.8 version. I recommend running both versions in windowed mode to allow switching between the two layouts to ensure that they match up.
introduce ModListEntry `offsetY` primarily to "fix" the alignment of the top-most Entry to align with 1.21.8's layout; and then each entry render's with the offset set. Set to 0 for no Y shift.
In order for the mods list to "align" with 1.21.8, I had to add a 2 px |
|
(I also couldn't open an issue on your fork) unsure of who's fault this is, but if this is of any help: |
That is a problem with the IMPL of the mod that you are clicking on; not Mod Menu's fault. I have tested this feature with MaLiLib.
|
oh alright 👍 |
|
Thank you updating this! I had small issue with 2025-10-02_17-46-31.mp4^ Chat Heads mod but also same issue applies to ClickThrough Plus as well |
The Pull Requests are not for Mod Support; See Issues |
|
awesome! |

I also sent a PR for PAPI for PatBox.