Skip to content
This repository has been archived by the owner on Jun 6, 2020. It is now read-only.

Find bar not following accent color #26

Open
Keith94 opened this issue May 16, 2018 · 8 comments
Open

Find bar not following accent color #26

Keith94 opened this issue May 16, 2018 · 8 comments
Labels
bug Something isn't working

Comments

@Keith94
Copy link

Keith94 commented May 16, 2018

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

@spikespaz spikespaz added this to the Version 2.0 (Rewrite) milestone May 16, 2018
@spikespaz spikespaz added the bug Something isn't working label May 16, 2018
@HRussellZFAC023
Copy link

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

@Keith94
Copy link
Author

Keith94 commented May 19, 2018

Yes, that's a pretty nasty bug!

@spikespaz
Copy link
Owner

@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.

@Keith94
Copy link
Author

Keith94 commented May 19, 2018

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

image

@HRussellZFAC023
Copy link

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

@martinguti95
Copy link

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

@martinguti95
Copy link

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.

@spikespaz
Copy link
Owner

@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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants