Skip to content

certus quartz tank item rendering improvements - #355

Merged
Dream-Master merged 6 commits into
GTNewHorizons:masterfrom
danyadev:certus-quartz-tank-item-rendering
Nov 9, 2025
Merged

certus quartz tank item rendering improvements#355
Dream-Master merged 6 commits into
GTNewHorizons:masterfrom
danyadev:certus-quartz-tank-item-rendering

Conversation

@danyadev

@danyadev danyadev commented Nov 8, 2025

Copy link
Copy Markdown
Member
  • render the tank model only once per renderItem call. Previously it tried to simulate glDisable(GL_CULL_FACE), now I'm just calling it explicitly. Now the bottom side renders correctly: it uses the bottom texture instead of the top one
  • disabled GL_BLEND for the tank, left it on for the fluid
  • disabled GL_LIGHTING for the fluid
Before After
image image

Changed it because I didn't like the sides of the fluid having a shade, and after disabling GL_LIGHTNING the tank became too dark compared to the fluid, so I disabled GL_BLEND too. Also now the item looks exactly the same as the block

  • Fixed fluid rendering in the tank if it is placed after a bucket

Before / After:
image
image

  • Fixed fluid rendering in the hand while in first person perspective

Before / After: (in both cases it had fluid, there were two separate bugs with its rendering)
image
image

  • Fixed item rotation when tossed in the world
Before After
image image

that way it simulated disabled GL_CULL_FACE for some reason. I changed it to actually disabling GL_CULL_FACE, so it renders the same, except that the bottom part is rendering the actual bottom side, not the top side
it makes the item render almost the same way as the block renders
@Dream-Master
Dream-Master requested a review from a team November 8, 2025 15:09
@Dream-Master Dream-Master added the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta (experimental build) label Nov 8, 2025
@Dream-Master
Dream-Master requested review from a team, Kogepan229 and lc-1337 November 9, 2025 11:08

@sisyphussy sisyphussy left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm

@Dream-Master
Dream-Master merged commit 8909e28 into GTNewHorizons:master Nov 9, 2025
1 check passed
@danyadev
danyadev deleted the certus-quartz-tank-item-rendering branch November 9, 2025 11:17
@Dream-Master Dream-Master removed the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta (experimental build) label Dec 13, 2025
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.

3 participants