Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added text of signs placed on all blocks that provide tabs to the tab tooltips #52

Draft
wants to merge 2 commits into
base: 1.19.3
Choose a base branch
from

Conversation

TheMrEngMan
Copy link

@TheMrEngMan TheMrEngMan commented Aug 18, 2022

The sign must be placed directly on the chest/barrel/shulker box etc. to show up in the tooltip.
Text color applied with dyes and custom formatting is also applied to tooltip text.
This works for double chests and chests identified with item frames as well.
image

Draft pull request because there is an overlap issue with the tooltips and status effects on the right of the inventory, simply changing the z-offset of the tooltips (by translating the MatrixStack in the z direction before drawing the tooltips) didn't seems to fix this issue...
image

… tooltips.

The sign must be placed directly on the chest/barrel/shulker box etc. to show up in the tooltip.
Text color applied with dyes and custom formatting is also applied to tooltip text.
This works for double chests and chests identified with item frames as well.
@Andrew6rant
Copy link
Owner

This is fantastic, sorry for the delay in reviewing this PR, I've been very busy these past few weeks.

I've noticed one small issue - the tooltip seems to snap strangely on double chests. Check it out:

Single Chest:

Minecraft_.1.19.-.Singleplayer.2022-09-04.15-22-02.mp4

Double Chest:

Minecraft_.1.19.-.Singleplayer.2022-09-04.15-22-22.mp4

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