Skip to content

Improved recipe viewer slots and general compat - #78

Open
Trinsdar wants to merge 85 commits into
brachy84:1.20.1from
Trinsdar:jei-rei-recipe-viewer-slots
Open

Improved recipe viewer slots and general compat#78
Trinsdar wants to merge 85 commits into
brachy84:1.20.1from
Trinsdar:jei-rei-recipe-viewer-slots

Conversation

@Trinsdar

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread src/main/java/brachy/modularui/integration/jei/JeiRecipeViewerSlot.java Outdated
Comment thread src/main/java/brachy/modularui/integration/rei/recipe/ModularUIREIDisplay.java Outdated
Comment thread src/main/java/brachy/modularui/integration/rei/ReiRecipeViewerSlot.java Outdated
Comment thread src/main/java/brachy/modularui/integration/rei/ReiRecipeViewerSlot.java Outdated
Comment thread src/main/java/brachy/modularui/integration/rei/ReiRecipeViewerSlot.java Outdated
Comment thread src/main/java/brachy/modularui/integration/jei/JeiRecipeViewerSlot.java Outdated

@screret screret left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

went and touched this up a 'bit', could you test the stuff now and make sure it works? Thanks!

Trinsdar and others added 10 commits August 1, 2026 23:51
…create a dummy recipe viewer slot widget impl
…st of the actual JEI slot's values with new ones.

this is done because JEI keeps track of all recipe slots & it's easier to replace existing slots' data than it is to replace the slots themselves.
…s in ModularUIRecipeCategory and ModularUIREIDisplayCategory to `getMaxWidth` and `getMax(Display)Height` respectively
…dd variants for JEI and REI too (so it isn't just EMI)
@Trinsdar Trinsdar changed the title Implementations of Jei and Rei recipe viewer slots Improved recipe viewer slots and general compat Aug 2, 2026
screret and others added 30 commits August 7, 2026 21:11
…e instead of keeping a global cache

Java garbage collection will take care of the rest, right?
…'d always just redraw existing elements from the screen we're embedded in
(i cant test this, sorry. laptop running out of battery.)
This reverts commit 79b44d5.
Reverted since the previous method of adding the tooltips made it
possible to modify said tooltips from the widget
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.

2 participants