Skip to content

Fix color lists - #349

Closed
ninjaguardian wants to merge 2 commits into
isXander:ltsfrom
ninjaguardian:fix/color-list
Closed

Fix color lists#349
ninjaguardian wants to merge 2 commits into
isXander:ltsfrom
ninjaguardian:fix/color-list

Conversation

@ninjaguardian

Copy link
Copy Markdown

Lists of colors were not correctly passing focus.
This also adds a list of colors to the test gui.

(It also fixes two stonecutter issues that were preventing me from compiling)

@ninjaguardian

Copy link
Copy Markdown
Author

The main branch has this issue too, but it's much more complicated. The fix in this pr does not work for it, and the 3 minecraft versions respond differently to fixes.

@ninjaguardian
ninjaguardian deleted the fix/color-list branch August 1, 2026 05:17
@ninjaguardian

Copy link
Copy Markdown
Author

I meant to rename it...
Oh well.
lts: #351
main: #350

I also separated out the stonecutter fix into its own pr: #352

@ninjaguardian

Copy link
Copy Markdown
Author

The main branch has this issue too, but it's much more complicated. The fix in this pr does not work for it, and the 3 minecraft versions respond differently to fixes.

In 26.x, ContainerEventHandler.super.setFocused(boolean) got an impl, which was causing issues.

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