SFX-WoW / Masque

A button-skinning engine for World of Warcraft.
Other
47 stars 11 forks source link

Dominos Masque appearance disappears after stealth or mount events #348

Closed BorisAneli closed 10 months ago

BorisAneli commented 10 months ago

Game Flavor

Classic Era

Game Version

1.15.0.52186

Add-On Version

10.2.0

Description

I am using Dominos addon for action bars. Masque has support for dominos addon but I noticed, that stealth / unstealth, mount / dismount events do reset masque on dominos action bars.

Looks like this events triggers some UI update functions which do not include mask logic for some reason

Steps to Reproduce

  1. Install dominos
  2. Install mask
  3. Do some mask settings that are apply to dominos
  4. Do mount / dismount / stealth / unstealth

Result: After event (4) dominos bars displays without mask

Expected result: Events does not affect appearance of dominos bars

Error Message

No response

Screenshots

https://github.com/SFX-WoW/Masque/assets/83017159/ebbbab44-9cb5-4523-8f8d-9c105e887253

Notes

No response

StormFX commented 10 months ago
BorisAneli commented 10 months ago

Yes, it's recolored dominos skin, problem can be in skin itself?

StormFX commented 10 months ago

Well, some skins use the default textures and that might obfuscate the issue in certain conditions. In order to analyze what's going on, you should try a skin that doesn't use the default textures to eliminate the skin being "reset" as a possible culprit. Try testing with a skin like Caith that doesn't use the default textures.

BorisAneli commented 10 months ago

Dominos skin looks perfect for me, all other skins have too modern look, so I wanna try to figure out whats going on, maybe Ill even try to fix it on my own. Can u please give me some clues about what's going on, why default textures can behave like this, does this logic on masque side or on dominos? As software engineer myself I see high probability of fix in one function call in correct place (maybe on dominos side)

StormFX commented 10 months ago

I wasn't asking you to use a different skin, I was asking you to try a different skin and see if the issue still occurs. This will give me more of an idea on where to look.

StormFX commented 10 months ago

You could try commenting out line 91 of Masque_Dominos/Skins.lua and see if that helps, though it may cause other issues. Eg:

From:

        UseStates = true,

To:

        -- UseStates = true,
BorisAneli commented 10 months ago

I checked out each default skin as well as caith skin Сaith as well as all default skins exept "Blizzard classic" do work without any issues

WIth default "Blizzard classic" skin initial issue occurs, so its not only "Masque: Dominos" skin issue

Also I tried to comment UseStates = true in "Masque: Dominos", nothing did change

StormFX commented 10 months ago

Ok. I'll have to look into this a bit further.

StormFX commented 10 months ago

It is as I suspected. In Classic/Classic era, the Normal texture is having its color changed by Blizzard code whereas in Retail, this color change is applied to the Icon instead. Commenting out the UseStates line does work, but you have /reload after you change it.

BorisAneli commented 10 months ago

Thanks a lot for help!