Skip to content
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

Make text more readable by using scrolling strings for bounded strings #8232

Merged
merged 17 commits into from
Oct 24, 2024

Conversation

pupnewfster
Copy link
Member

Changes proposed in this pull request:

Replaces our concept of text scaled based on bound (which can make text tiny and impossible to read), with using mojang's scrolling text system

… scrolling if it ends up going out of bounds
…ady display the information in the neighboring screen and the tooltip feels clunky
…so that we only have to define the max text bounds in one place
… scrolling text for default configs when set to english
…to make use of scrolling text for default configs when set to english
… that the dictionary's dropdown doesn't have to have scrolling text in english (currently still does for BE Types)
…not quite translating to the correct spot (seems to make it slightly more readable)
…ionary dropdowns fit within the gui in english without resorting to having to scroll
@pupnewfster pupnewfster marked this pull request as ready for review October 10, 2024 19:04
…string depending on LTR or RTL language instead of starting at a random position
…, and further improve when scrolling text gets reset in relation to switching to recipe viewers and resizing MC
@pupnewfster pupnewfster merged commit 3fea841 into 1.21.x Oct 24, 2024
3 checks passed
@pupnewfster pupnewfster deleted the scrolling_strings branch October 24, 2024 16:15
@pupnewfster pupnewfster added this to the 10.7.6 milestone Oct 24, 2024
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.

1 participant