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

Change to INameable to grab custom name or block name, and highlight fix #64

Open
wants to merge 2 commits into
base: 1.16
Choose a base branch
from

Conversation

DatenThielt
Copy link

@DatenThielt DatenThielt commented Apr 19, 2022

To display custom names instead of default block names, Changed method of returning displayName to lookup iNameable and use getDisplayName over getHoverName.

I have removed what appears to be some method of returning names on Modular storages as it does not work, Also modular storages do not seem to retain their NBT when being placed so naming them in an anvil does not work, unfortunately I have not had a chance to look at why that might be,

Also I have included a fix to update the blockHighlight on double click

To display custom names instead of default block names, Changed method of returning displayName to iNameable and use getDisplayName over getHoverName.
Fixed block highlighting by utilizing the newer RFToolsBase.highlightBlock method.
@DatenThielt DatenThielt changed the title Change to INameable to grab custom name or block name Change to INameable to grab custom name or block name, and highlight fix Apr 19, 2022
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.

1 participant