Accessibility Feature#128
Open
armele wants to merge 1 commit into
Open
Conversation
Contributor
|
In this UI specifically, can't we set a black font on the itemicon? |
Author
|
From the XSD what I saw was that only has item, nbt, properties, and renderItemDecorations; no color/textcolor... (Didn't see anything in the ItemIcon classes or nearby that would contradict that, either.) We could theoretically add plumbing to allow a color to be specified at design time (maybe at the BOWindow level?) but I feel like that would be even more heavy-handed, while also not allowing the player to decide if they need the higher contrast rendering. The advantage is that it would be per-window rather than literally every ItemIcon... I will of course defer to your wishes! |
Contributor
|
Do not trust the XSD |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Closes Discord Suggestion
Changes proposed in this pull request
AI
I did use AI on this feature - both to understand the current functionality, and to implement (and then review) the changes. I did review every line of changes to ensure I had a solid understanding of the purpose.
Example
Testing
Review please