Skip to content

One tooltip in MV#189

Merged
Alatyami merged 3 commits into
GrowthcraftCE:developmentfrom
moonfather1:cloth_tooltip
Jul 11, 2026
Merged

One tooltip in MV#189
Alatyami merged 3 commits into
GrowthcraftCE:developmentfrom
moonfather1:cloth_tooltip

Conversation

@moonfather1

@moonfather1 moonfather1 commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Added a tooltip below result in MV telling us how to get the item out.

Same functionality as in 1.20.1. More problematic to implement as client menu can no longer access block entity.

Closes #151

@Alatyami

Copy link
Copy Markdown
Member

@moonfather1 completeRecipe() puts recipe.getResultActivationTool() into the new real container slot. When the Mixing Vat is broken, MixingVatBlock.onRemove() calls Containers.dropContents(...), which drops every item in the container—including an extra cheesecloth. We'll need to exclude the item from drops.

Screenshot 2026-07-11 104622 Screenshot 2026-07-11 104559

There's also a duplicte tool tip that triggers from the player inventroy.

image

@moonfather1

Copy link
Copy Markdown
Contributor Author

Yep. Very good catches on both of those.
I'll get back to it.

@moonfather1

Copy link
Copy Markdown
Contributor Author
  • Fixed screen issue. Just look at that diff...
  • Fixed unintended cloth drop.
  • Fixed a related exploit - breaking MV shouldn't be used to circumvent cloth requirement.

Comment thread src/main/java/growthcraft/milk/block/MixingVatBlock.java
@moonfather1

Copy link
Copy Markdown
Contributor Author

Good catch again. We should be good now.

@Alatyami Alatyami merged commit 109391d into GrowthcraftCE:development Jul 11, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Restore Mixing Vat cheese cloth tooltips

2 participants