spikespaz/firefox-nativedark

Find bar not following accent color

Keith94 opened this issue · 8 comments

I chose this accent color in Windows,

applicationframehost_2018-05-15_21-29-13

And my findbar at the bottom looks like this as a result (is there a way to follow accent color?)

firefox_2018-05-15_21-24-02

image
in addition, if the difference mode is set to lighter, the findbar is unreadable.

Yes, that's a pretty nasty bug!

@HRussell63250 I think that this is an issue related to not setting the color for the bottom bar. Did this only start happening after the FF 60 update? It appears to be adding a white overlay to a transparent or white background. I think I may be able to fix this, I'm not sure though.

Fun fact: If Unfocused theme is set to Light titlebar, findbar is displayed correctly only when inactive.

image

Yup. I think it is related to the new theme API introduced:
https://developer.mozilla.org/en-US/Firefox/Releases/60

Hi,

I'm assigning this to me. I've been able to replicate what @Keith94 raised about the Unfocused theme and the Light Titlebar and I think that there's another style overriding the value of the accent color when focused

Checking the release notes of FF 61 https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/61#WebExtensions , now the toolbar attributes apply also to the find bar.

When the window is selected, a default value is applied to the toolbar. That's what is causing the bug.

@martinguti95 Yes, and that snippet of code in one of the commits solves the issue, however there are problems with it as you will find out. If you see another way to fix this, I'm all ears.