Skip to content

Conversation

@Zorbatron
Copy link
Member

What

In #2672, I accidentally made it so if a GT fluid slot had no fluid, it would insert an empty key into the tooltip instead of doing nothing which made it so empty slots would have a small black dot near the cursor.
Additionally makes it so these only happen when the fluid actually changes:

  • rebuilding the tooltip
  • syncing the fluid server -> client

@Zorbatron Zorbatron requested a review from a team as a code owner September 7, 2025 17:24
@Zorbatron Zorbatron added type: bug Something isn't working type: refactor Suggestion to refactor a section of code labels Sep 7, 2025
if (a == b) return true;
if (a == null) return false;
return a.isFluidEqual(b);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Why make this when it is covered by FluidStack#isFluidEqual()?

Copy link
Member Author

Choose a reason for hiding this comment

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

Because it replaces if (init || current == null || lastFluid == null || current.isFluidEqual(lastFluid)) { so it works when both stacks are null.

@Zorbatron Zorbatron merged commit 99d244a into master Sep 15, 2025
3 checks passed
@Zorbatron Zorbatron deleted the zb/fluid-slot-tooltip-empty-fix branch September 15, 2025 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug Something isn't working type: refactor Suggestion to refactor a section of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants