GoldpawsStuff / AzeriteUI5

Custom user interface suite for World of Warcraft. Design and code by Lars Norberg and Daniel Troconis.
https://www.patreon.com/GoldpawsStuff
Other
22 stars 4 forks source link

[Feature] Make Explorer Mode respect action bar fade setting for fully faded bars set to fade separately #54

Closed anaarkei closed 7 months ago

anaarkei commented 7 months ago

Game Flavor

Dragonflight

Is there an existing issue for this?

Describe the bug

Occurs if Explorer Mode is enabled and all fades are selected, and ”Don’t fade in other action bars” is selected on an action bar.

Hovering the mouse over faded action bar (in this case, Action Bar 3) to make it display causes other action bars to appear (in this case, Action Bar 1 and the attached Player UnitFrame).

To reproduce

Enable Explorer Mode with all fades selected

Select ”Don’t fade in other action bars” on an action bar other than Action Bar 1 or 2.

Expected behavior

Action Bar 3 should be the only thing that appears on mouseover.

Screenshots

Screenshot 2023-12-11 at 9 55 26 pm Screenshot 2023-12-11 at 9 55 14 pm

Additional context

None

Goldpaw commented 7 months ago

That's actually not a bug.

You haven't set the entirety of bar 1 to fade out, you appear to have the default setting where only the last 5 buttons are set to fade. And they indeed remain faded out, unaffected by you hovering over bar 3. If you set bar 1 to fully fade out in the actionbar settings, it will remain faded when you hover over bar 3.

While having the option to fade out actionbars set in the Explorer Mode settings, hovering over any actionbars will force the Explorer Mode to exit, showing things as though it wasn't even there. Which it does.

So this is more of a feature request. I think a solution that would get us where we want here without having to change any of the options is to simply exempt actionbars that are set to fully fade (all buttons) from the explorer mode. So their fading is handled only by the actionbar settings, not affecting the frames handled by the explorer mode.

Goldpaw commented 7 months ago

Fix/implemented in 5.2.140-RC. Working as we want now!