-
Notifications
You must be signed in to change notification settings - Fork 82
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
Update to 1.20.4 #500
base: multiloader/1.20.X
Are you sure you want to change the base?
Update to 1.20.4 #500
Conversation
It appears that there are still some issues with FabricClientHelper/ForgeClientHelper, I'll probably take a closer look at those tomorrow |
I'd just like to let you know about a crash this causes when opening the bindings in the menu, here's the crash report: The game crashed whilst rendering screen The full crash report is here: https://pastebin.com/JqHv936F Thanks for all your work updating it! |
This crash (and various similar ones) should be fixed now. I replaced the usages of reflection that were causing this with access transformer entries, so this will be noticeable during compile time if it breaks again in a future update. |
The menu to rebind keys on fabric at least is very dark, and it doesn't even allow you the option to add external bindings but it lets you see them, unlike before where you could add them. |
The dark menu should be fixed now, it appears that the background gradient was already being drawn by the parent class. |
I'm interested in adding NeoForge support to this PR, but I cannot get the maven repository for Framework to behave. Is there some important setup I'm missing? |
You need to add personal access token with read:packages scope and then set GPR_USER=your@email and GPR_KEY=token environment variables or gpr.user=your@email and gpr.key=token in gradle.properties... |
Ah my mistake, I was using my github username not e-mail, in combination with, I assume, regenerating the token but not saving it. Thanks |
@JulianVennen How did you solve the issue with AbstractContainerScreen.slotClicked being changed from protected to public through the Access Transformers? I'm getting errors about it's subclasses having weaker access privileges. |
You need to reload the Gradle project in your IDE every time you modify the access transformer. You probably also need to tell neogradle the path to your access transformer if you are talking about your neoforge project. |
This PR updates Controllable to Minecraft 1.20.4
Both Fabric and Forge have been tested and work correctly.
Please note that the ForgeGradle runs appear to be broken, causing a crash in the development environment.
Compiling a jar and running it in a normal Minecraft instance is unaffected.
If supporting NeoForge as well is desired, please let me know and I will implement that too.