-
Notifications
You must be signed in to change notification settings - Fork 424
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
Proposal to Change Creative Buttons Texture #4221
base: 1.21.3
Are you sure you want to change the base?
Proposal to Change Creative Buttons Texture #4221
Conversation
I believe it makes a lot more sense to have secondary buttons there instead of primary ones. In-game UI mostly doesn't have primary buttons (except for the |
I like the new look, but the right edge of the buttons doesn't align with the right edge of the scrollbar. To be fair this was also an issue with the previous implementation, but it seems more obvious now. |
I like the first revision, yes. |
I think this looks great 👍 |
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.
Im happy with this, its a definite improvement. If anyone thinks they can be done better again please let it be known as I dont plan on touching this yet agian.
Only issue I can think of is that by trying to fit into the vanilla scheme here the contrast on the active buttons end up being low. That may not be an issue due to the physical location but I believe it is something that should still be thought about. |
Alright, I'm thinking about the solutions, and here are the options,
I'm not really sure what to do, but I think probably move the buttons back |
I actually think shortening the search bar would look better. |
@matthewperiut there's a fourth option, replace the outline with a normal cell border on the search tab. |
Shortening the search bar would be perfectly acceptable in my opinion, users won't notice and it's aesthetically pleasing. |
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.
Shorten the search bar, as per consensus. Making a review of this so it doesn't get forgotten when last call ends.
Yeah sorry guys work got busy, let me try and get this finished |
Ok, now it is ready (I think) |
@@ -68,11 +73,11 @@ private void updateSelection() { | |||
} | |||
} | |||
|
|||
@Inject(method = "init", at = @At("RETURN")) | |||
@Inject(method = "init", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/gui/widget/TextFieldWidget;setEditableColor(I)V", shift = At.Shift.AFTER)) |
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.
This was changed to stop creating creative buttons when in survival mode, incase anyone asks the purpose
… gui and code quality
Looks great IMO 👍 |
…bility with resource packs
Me thinking I'm done, having random thoughts of what ifs and how to fix them later... guys maybe its truly ready now. |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
I decided not to mess with the png size as that is how it was originally, I am truly solely trying to get the design of the button to be different and making sure the code I add isn't inefficient. But yes, in the current version widgets are usually just small images with a different image loading system. |
Screenshots of updated texture and position
BELOW IS OUTDATED!
tl;dr
video.mp4
Proposal to Change Creative Buttons Texture
The creative buttons texture has haunted me for years in fabric-api. Why?
They're blue. The entire interface for buttons hasn't been blue in over 6 years.
Additionally they have too much contrast, sticking out like a sore thumb in the user interface.
How do we fix it?
We have a great example of something interactable in the creative menu already, the scroll bar.
It has an inactive state and an active state:
active:
inactive:
Extrapolating the Features
Notice that the creative scroll bar is in an indented section where it can move freely.
This creates a contrast strong enough to differentiate it from the rest of the menu.
The "grip" lines horizontal on the bar also changes color depending on if you can or cannot scroll.
Using the Features
Including the indent to seperate it from the rest of the UI helps the contrast to make it visible, without making it stand out visually from the slots and the scroll bar.
Using the pallette of the scroll bar in its active and inactive state for the pallette in the buttons being active and inactive makes it more flush with the design and not stick out.
Result:
Default appearance:
Hovering on right:
Hovering on left:
Disabled left:
Disabled right:
Full screenshot view of change
Previous appearance for reference:
Small Feats
Reduced color pallete allows for greater compression
5,512 bytes -> 382 bytes
14x decrease in size
Jar size improvement:
1,992,580 -> 1,989,946
0.13% total jar size improvement
(small things matter LOL)
detail: compiled on arch linux with Temurin-21.0.5+11-LTS
Footnote additional details
The arrow is made white instead of the button outline because it becomes too noisy in a small area.
Minecraft 1.15 removed blue backgrounds from buttons in Minecraft, and it came out in 2019, 6 years ago.
Previous attempts, which the textures were worse rip- #2740, #2746