Skip to content

Allow drawing non-living entities with EntityDrawable (and port f61dc8e)#4065

Merged
YoungOnionMC merged 18 commits intoGregTechCEu:mui2-refactorfrom
screret:sc/mui/entity-render-things
Feb 3, 2026
Merged

Allow drawing non-living entities with EntityDrawable (and port f61dc8e)#4065
YoungOnionMC merged 18 commits intoGregTechCEu:mui2-refactorfrom
screret:sc/mui/entity-render-things

Conversation

@screret
Copy link
Copy Markdown
Contributor

@screret screret commented Oct 12, 2025

What

Ported f61dc8e & and did some formatting tweaks
also removed render state calls that ended up doing nothing
and disabled spotless on the test ui layout code because it added annoying line breaks

Implementation Details

The code for drawing entities is mostly copied from InventoryScreen#renderEntityInInventory with minor tweaks (which should be commented)
The render state modification calls should probably? be checked for if they actually did do something. I believe they didn't, though.

Outcome

can render item frames, boats, etc. non-living entities with EntityDrawable

@screret screret requested a review from a team as a code owner October 12, 2025 15:11
@YoungOnionMC YoungOnionMC added type: bugfix General bug fixes Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. labels Oct 12, 2025
# Conflicts:
#	src/main/java/com/gregtechceu/gtceu/api/mui/drawable/text/StyledText.java
#	src/main/java/com/gregtechceu/gtceu/common/machine/muimachine/TestMuiMachine.java
# Conflicts:
#	src/main/java/com/gregtechceu/gtceu/api/mui/drawable/GuiDraw.java
#	src/main/java/com/gregtechceu/gtceu/api/mui/drawable/SpriteDrawable.java
#	src/main/java/com/gregtechceu/gtceu/api/mui/drawable/text/TextRenderer.java
#	src/main/java/com/gregtechceu/gtceu/client/mui/screen/viewport/GuiContext.java
#	src/main/java/com/gregtechceu/gtceu/common/machine/muimachine/TestMuiMachine.java
@screret
Copy link
Copy Markdown
Contributor Author

screret commented Jan 6, 2026

pls merge?

@screret
Copy link
Copy Markdown
Contributor Author

screret commented Jan 31, 2026

pls merge?

@YoungOnionMC YoungOnionMC merged commit 4fd1c42 into GregTechCEu:mui2-refactor Feb 3, 2026
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. type: bugfix General bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants