-
-
Notifications
You must be signed in to change notification settings - Fork 215
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
Improvement: Enchant Parsing SBA Convert Roman Numerals and NEU Inventories compat #1717
Conversation
ItemHoverEvent already does the same thing. just make its list modifyable, ig |
Oh I see, idk how I missed that event. Yeah I'll probably just use that and make it modifiable. |
Doesn't work with NEU enchant colors |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mikecraft1224's comment
This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes. |
# Conflicts: # src/main/java/at/hannibal2/skyhanni/features/misc/items/enchants/EnchantParser.kt
Conflicts have been resolved! 🎉 |
Moving the ItemHoverEvent mixin inject ensures it is never affected by NEU enchant colors so it doesn't break and die. If anyone tests this make sure your repo is using the updated regex otherwise the tooltip will be missing random enchants. Works with roman numerals to numbers from both sba and sh, but the roman to numbers from sh doesn't work in NEU pv, but I think thats for @Mikecraft1224 to amend. |
This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes. |
Conflicts have been resolved! 🎉 |
This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes. |
Conflicts have been resolved! 🎉 |
…tories compat (hannibal002#1717) Co-authored-by: Cal <cwolfson58@gmail.com>
What
Makes enchant parsing work when SBA's Convert Roman Numerals feature is enabled and in NEU's inventories, such as /pv and storage overlay.
This will most likely need testing before approval since there is probably a certain case where my regex or code doesn't like a certain enchant format on some item / enchant / lore description. I've tested pretty much every item I have and other peoples items through /pv and they all work, but more testing from others would be appreciated.
Changelog Improvements
/pv
and storage overlay.