Skip to content

Conversation

@sakura-ryoko
Copy link
Contributor

I also sent a PR for PAPI for PatBox.

@sakura-ryoko sakura-ryoko changed the title [1.21.9]: Port to 1.21.9-pre1 [1.21.9]: Port to 1.21.9-pre2 Sep 19, 2025
@sakura-ryoko sakura-ryoko changed the title [1.21.9]: Port to 1.21.9-pre2 [1.21.9]: Port to 1.21.9-pre4+ Sep 24, 2025
@sakura-ryoko
Copy link
Contributor Author

This port should be solid for Release time.

Copy link

@NotRyken NotRyken left a 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.

@sakura-ryoko
Copy link
Contributor Author

See if the changes satisfies the requests.

@sakura-ryoko sakura-ryoko changed the title [1.21.9]: Port to 1.21.9-pre4+ [1.21.9]: Port to 1.21.9-rc1+ Sep 28, 2025
Copy link

@NotRyken NotRyken left a 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.

Copy link
Contributor

@haykam821 haykam821 left a 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]>
@sakura-ryoko
Copy link
Contributor Author

sakura-ryoko commented Sep 30, 2025

@haykam821 -- See the changes. There seems to have been a bug with how the DescriptionList was getting updated; by doing a simple if (mod != selectedMod) {} check.

@haykam821 haykam821 changed the title [1.21.9]: Port to 1.21.9-rc1+ Update mod to Minecraft 1.21.9 Sep 30, 2025
Copy link
Contributor

@haykam821 haykam821 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The selection highlight for parent mods is misaligned:

Misaligned selection highlight

In addition, expanding the children of a parent mod incorrectly scrolls the selected mod into view, and child mods are moved more to the right than before.

Copy link
Contributor

@haykam821 haykam821 left a 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.
@sakura-ryoko
Copy link
Contributor Author

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.

In order for the mods list to "align" with 1.21.8, I had to add a 2 px yOffset, and revert the text line changes.

@adr278
Copy link

adr278 commented Oct 2, 2025

(I also couldn't open an issue on your fork) unsure of who's fault this is, but if this is of any help:
I crashed while clicking inside Dynamic FPS' tab
I was using this: https://github.com/sakura-ryoko/ModMenu/actions/runs/18168097762
logs: https://mclo.gs/wR5kEYh

@sakura-ryoko
Copy link
Contributor Author

sakura-ryoko commented Oct 2, 2025

(I also couldn't open an issue on your fork) unsure of who's fault this is, but if this is of any help: I crashed while clicking inside Dynamic FPS' tab I was using this: https://github.com/sakura-ryoko/ModMenu/actions/runs/18168097762 logs: https://mclo.gs/wR5kEYh

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.

PressableWidget.onPress() is 'Missing'

@adr278
Copy link

adr278 commented Oct 2, 2025

(I also couldn't open an issue on your fork) unsure of who's fault this is, but if this is of any help: I crashed while clicking inside Dynamic FPS' tab I was using this: https://github.com/sakura-ryoko/ModMenu/actions/runs/18168097762 logs: https://mclo.gs/wR5kEYh

That is a problem with the mod that you are clicking on; not Mod Menu's fault. I have tested this feature with MaLiLib.

oh alright 👍
thanks for letting me know, much appreciated for the update too ❤️

@danielwerg
Copy link

Thank you updating this!

I had small issue with + sign and whatever element it belong too, not sure if this is mod menu issue (nothing happens when you click +)

2025-10-02_17-46-31.mp4

^ Chat Heads mod but also same issue applies to ClickThrough Plus as well

@sakura-ryoko
Copy link
Contributor Author

Thank you updating this!

I had small issue with + sign and whatever element it belong too, not sure if this is mod menu issue (nothing happens when you click +)
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

@Prospector
Copy link
Member

awesome!

@Prospector Prospector merged commit c08eeef into TerraformersMC:1.21.6 Oct 2, 2025
1 check passed
@sakura-ryoko sakura-ryoko deleted the 1.21.9-test branch October 2, 2025 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants