Implement a better options menu #42
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Bixilon
requested changes
Dec 9, 2025
Owner
Bixilon
left a comment
Bixilon
left a comment
There was a problem hiding this comment.
Just the initial code review, have not looked deeper inside nor tested it.
...in/java/de/bixilon/minosoft/gui/rendering/gui/elements/input/button/AbstractButtonElement.kt
Outdated
Show resolved
Hide resolved
...in/java/de/bixilon/minosoft/gui/rendering/gui/elements/input/button/AbstractButtonElement.kt
Outdated
Show resolved
Hide resolved
...in/java/de/bixilon/minosoft/gui/rendering/gui/elements/input/button/AbstractButtonElement.kt
Outdated
Show resolved
Hide resolved
|
|
||
| private companion object { | ||
| val CLICK_SOUND = minecraft("ui.button.click") | ||
| val CLICK_SOUND = "minecraft:ui.button.click".toResourceLocation() |
Owner
There was a problem hiding this comment.
val CLICK_SOUND = minecraft("ui.button.click")
Author
There was a problem hiding this comment.
I believe theres some issues with sounds, I couldnt get them to work, no ideas why...
src/main/java/de/bixilon/minosoft/gui/rendering/gui/elements/input/slider/SliderElement.kt
Outdated
Show resolved
Hide resolved
...main/java/de/bixilon/minosoft/gui/rendering/gui/gui/screen/menu/options/CloudSettingsMenu.kt
Outdated
Show resolved
Hide resolved
...main/java/de/bixilon/minosoft/gui/rendering/gui/gui/screen/menu/options/CloudSettingsMenu.kt
Outdated
Show resolved
Hide resolved
| updateDisabledStates() | ||
| } | ||
|
|
||
| private fun updateDisabledStates() { |
Owner
There was a problem hiding this comment.
The profile provides delegates. You can easily observe it. Justs create a profile synced button (or so), that observes the profile and updates the button when that "event" is triggered. No need for polling.
src/main/java/de/bixilon/minosoft/gui/rendering/gui/gui/screen/menu/options/OptionsMenu.kt
Outdated
Show resolved
Hide resolved
src/main/java/de/bixilon/minosoft/gui/rendering/sky/clouds/CloudRenderer.kt
Outdated
Show resolved
Hide resolved
Author
|
Slider bars still are wonky, I will rewrite that. |
Author
|
well that rebase fucking died on me. |
…fullbright. Theres currently many issues but I taped it together and it kinda works(?) it currently has these problems: * Doesnt include all options * Cloud options when changed override "moving clouds" setting. * Sliders sometimes dont set to exact numbers D: * Slider system isnt exactly same as minecrafts, which I am not sure how to make work... It doesnt feel as smooth as minecrafts sliders... * The Chat width and height text boxes cant be changed, the boxes needs a special input block similar to... you guessed it! * The tooltips dont follow the cursor, I couldnt find out a way to implement it that isnt a taped together ugly tech. * Sliders are missing the grayed out textures, currently only making them slightly transparent, at least til I can bother to fix the "knob" texture being an actual knob :') * Settings tabs should probably be reorganized in a better manner, current organization feels a little all over the place. * Translation strings needs to be added, I gave up at some point :') * I dont have headphones with me... Does the master volume slider work?? * Graying out of buttons doesnt work after rebasing. * Hi, please dont be mean :) Update SliderElement.kt fixed the updated version of minosoft (graying out buttons is broken)
fixes to make it actually compile, damn it.
Slider bars still are wonky, I will rewrite that. Controls are still not working, I am unsure how can I make something that looks good with current UI and menus. Do we... even support resource packs lmao?
…t and mouse capturing for it.
my bad for using outdated file haha :')
…n can be fullfilled This excludes cases if a combination for "E+T+Y" exists and you press "E" to open the inventory, you can press Y+T+E on those cases so neither chat or inventory opens and you can use the combination for that... I think even vanilla minecraft bothered with their controls menu this much I swear...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently doesnt have a menu for resource packs.
Theres an issue with updating the clouds when you change settings, I dont think its an issue on options menu side.
Its missing some options that would be nice to have, open for suggestions on that.
Chat settings should have a proper textbox implementation for setting width...