GTNewHorizons / GT-New-Horizons-Modpack

A big progressive questing modpack for Minecraft 1.7.10 balanced around the mod GregTech.
https://www.gtnewhorizons.com/
Other
1.02k stars 311 forks source link

[Reminder] Fix Bookmark side to account for PotionEffect UI in inventory #8469

Closed Shibva closed 2 years ago

Shibva commented 3 years ago

Your GTNH Discord Username

... its obvious at this point, Doggo

Your Pack Version

2.1.1.0

Original Request

A fix to the Inventory UI so the Bookmakrs tab accounds for Pottion Effect UI

Responding developer

@mitchej123#0313

Context

https://discord.com/channels/181078474394566657/181078474394566657/881248621725634630

Developer's Promise

He said that if I locate the section in JEI that allows for its own bookmark bar to consider any PotionEffect and to change its shape to acomidate for it accordingly that he will implement it

ref image for issue image

Final Checklist

Shibva commented 3 years ago

Ive also found this out image

man the potion ui is being an ass XD

Shibva commented 3 years ago

hope that this gets implimented as the next fix, a reminder, the context of some info I found on how to hande Potion UIs can be found here in the 1.12.2 version

I rememer being told that it will more likely to happen if I were to locate were in the 1.12.2 version of JEI on where the potion ui was handled with to be looked at by the person that is mantining the modified version of nei (person who is assigned to this)

To make things a bit easier to trace back, heres a link to the part of JEI's 1.12.2 code where the Bookmark bar considerds the potion UI

https://github.com/mezz/JustEnoughItems/blob/91995dd6d27056a0117159d5827c4923405dd4a6/src/main/java/mezz/jei/plugins/vanilla/InventoryEffectRendererGuiHandler.java

hope this makes things a bit easier then going back and forth between discord to get this link :)

github-actions[bot] commented 2 years ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 3 days