Mica is broken when switching light/dark theme about firefox-uwp-style HOT 19 CLOSED

You can go ahead.

from firefox-uwp-style.

Mica material is applied externally, Firefox does not call any of the apis that activate it or decide what theme to use. If you want to change it while it is open do it through MicaForEveryone.

from firefox-uwp-style.

Oh, yes you're right, I've missed that options (was set to Default). Switching that together with the Firefox theme works great!

Thank you.

Closing...

from firefox-uwp-style.

@Guerra24 I have to reopen this since there is an unfixed issue. When the theme changes and firefox is closed, the next reopening is still the previous Mica theme.
See this issue for more details: MicaForEveryone/MicaForEveryone#64

from firefox-uwp-style.

When the theme changes and firefox is closed, the next reopening is still the previous Mica theme.

Works here. MFE's Firefox rule is set to System.

from firefox-uwp-style.

🤔 I already have the same rule but the issue persists...

from firefox-uwp-style.

@Guerra24 I can also reproduce the issue, however not in the order that you shown in the video, maybe do in this order instead:

1. Open Firefox. Confirm that the correct theme (dark/light) is rendered.
2. Close Firefox.
3. Change the theme
4. Reopen Firefox. Confirm that it renders the wrong theme.

In your video, you changed the theme before closing Firefox, which doesn't seem to trigger this bug.

EDIT: Reproducing video

from firefox-uwp-style.

Well, that's acrylic... afaik it does not have theme-specific colors. Also since I'm running stable Win11 I have not tested any of the other backdrop types. Regardless, I tried and still works just fine.

That aside this is something I can't fix from my side unfortunately.

from firefox-uwp-style.

The same happens with Mica actually (sorry for the horrible video btw). And also looks like we are running dev builds while you are running stable. I generally think this is a bug related to Firefox, since this behavior doesn't shown on any other apps, only Firefox.

.

from firefox-uwp-style.

Closing this since the issue is unfixable from my side.

from firefox-uwp-style.

I've found a workaround for this issue: I've written an AutoHotKey script that automatically triggers reload when theme has changed and Firefox window is active (or is opened):
https://github.com/maicol07/AHK-scripts/blob/main/FixFirefoxMFE.ahk

Feel free to improve the script if you want!

from firefox-uwp-style.

Just upgraded to 22621 and so far no issues on my side in both stable and nightly.

from firefox-uwp-style.

Strange, it still happens on my machine (25140). It looks like Firefox is automatically changing DWMWA_USE_IMMSERIVE_DARK_MODE, because when I delay execution of DwmSetWindowAttribute, everything works perfectly.

from firefox-uwp-style.

After some Google digging, this problem is most likely caused by Firefox.
https://bugzilla.mozilla.org/show_bug.cgi?id=1734359
https://hg.mozilla.org/integration/autoland/rev/b94153cca1f7

EDIT: The title bar bug happens even without MFE, or any other program messing with Firefox's immersive dark mode attributes. So, it's definitely a Firefox bug.

EDIT 2: Firefox uses a function within the LookAndFeel class in order to determine whether it should use dark mode for the title bars. In the LookAndFeel.h file, the function returns the value of a field. Haven't managed to find out where the field is set though, nor the source file of the LookAndFeel class (commonly it would be LookAndFeel.cpp, but the file is missing).

EDIT 3: Looks like this function is to blame: https://github.com/mozilla/gecko-dev/blob/master/widget/LookAndFeel.h#L391.

from firefox-uwp-style.

Btw are you all sure Titlebar color is set to System in MFE? I can replicate the issue when set to Default but with System it does work just fine.

from firefox-uwp-style.

Btw are you all sure Titlebar color is set to System in MFE? I can replicate the issue when set to Default but with System it does work just fine.

Yes. And as I said before, it does not matter whether MFE is used or not. This is a bug within Firefox, not MFE or this theme.

from firefox-uwp-style.

Whether MFE works or not in this case is pretty much a hit or miss. Some systems have Firefox applying dark theme before MFE and doesn't cause this bug; while others have MFE applying dark theme first, leading to this bug.

from firefox-uwp-style.

On my system, Firefox takes like 3 seconds to initialize, while MFE only needs like 1 for applying the title bar color to Firefox. Yours looks pretty much instant.

from firefox-uwp-style.

Okay, so this is the issue. Firefox theme detection is somehow off by one and since it's a bug in Firefox it can be reported through bugzilla and it should be valid since the issue is visible when the titlebar is enabled. I can do it but I'm not sure when, if you want to do it just let me know otherwise some time next week I will.

from firefox-uwp-style.