Skip to content

Partially fixes the ui bug with the rock sorter recipes#641

Open
frogicillin wants to merge 2 commits into
SymmetricDevs:mainfrom
frogicillin:local-frog
Open

Partially fixes the ui bug with the rock sorter recipes#641
frogicillin wants to merge 2 commits into
SymmetricDevs:mainfrom
frogicillin:local-frog

Conversation

@frogicillin

Copy link
Copy Markdown

Reflows item output slots for the ore sorter recipes that would overflow the JEI panel. Unfortunately couldn't fix the slot background yet. Input slots are passed through unchanged. Done as a mixin for gregtech rather than JEI because I didn't want to mess with JEI lol.

@trainvoi trainvoi added the fix A fix to a bug (used for PRs) label May 23, 2026
@trainvoi

Copy link
Copy Markdown
Contributor

is there a screenshot of your fix in action?

@bruberu bruberu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image I do not currently notice any appreciable difference.

@frogicillin

frogicillin commented May 26, 2026

Copy link
Copy Markdown
Author

What recipe is this? Why is iron an input in the ore sorter creating copper? I can't recreate this to fix...

The intended behavior is changing ore sorter recipes from this:
image
To this:
image

@bruberu

bruberu commented May 27, 2026

Copy link
Copy Markdown
Member

Wait I actually cannot see the difference.
Also the copper thing is a test recipe, don't worry.

@frogicillin

frogicillin commented May 27, 2026

Copy link
Copy Markdown
Author

The difference is the items in the fifth column are shifted down to the lower rows. Problem was that you couldn't hover over any of the items in the fifth column because there was some weird interaction with the JEI menu. Could not figure out how to remove the columns background at the time so that's why I called it a partial fix.

@bruberu

bruberu commented Jun 5, 2026

Copy link
Copy Markdown
Member

Oh that does make sense. I suppose a better way to actually deal with it though is to shift the UI elements over to the left, although more code would be needed for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix A fix to a bug (used for PRs)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants