Mild incompatibility with Just Enough Effect Descriptions
aaronhowser1 opened this issue ยท 5 comments
This video actually seems to showcase three separate issues.
- The first one, and the one EMI can trivially fix: stacks are rendering tooltips in generic exclusion area (on pages other than the first), visible in the video. This is a simple logic error made by some changes in sidebar calculations, and I'll be sure to fix it.
- The second one, which is likely outside of EMI's control, is the tooltips rendered where effects aren't. EMI makes sure to diligently reuse vanilla codepaths in its mixin for moving effects around, and it appears JEED's effect location calculation is naive, which will either require an adjustment of logic on their side, or integration with EMI (using implementation classes, which is undesirable)
- The third one, potentially outside of EMI's control and more minor, is the double tooltip when hovering compressed effects. In vanilla, hovering a compressed effect (which happens on thin screen sizes) causes the effect to display a tooltip of remaining time. It may be possible EMI's method of compression is not being detected by JEED, or JEED may simply not recognize this vanilla behavior. This would also be resolved by JEED being more particular with its location detection.
I believe I'll be closing this issue when I've resolved and released the first of three listed, and will open an issue on JEED regarding the latter two.
I worked together with MehVahdJukaar and wrote a mixin for them which has better detection for both EMI and any mods that have other functions like this, this version of JEED has been released. Including some local changes, I've been able to confirm it works perfectly with EMI's settings now.
This solves the second and third issues, and I'll plan a release later today resolving the first, kinda shocked with how fast this collaborative turnaround went ๐