Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make browser icon more visible with dark browser themes #46

Closed
Decopi opened this issue May 14, 2018 · 3 comments · Fixed by #179
Closed

make browser icon more visible with dark browser themes #46

Decopi opened this issue May 14, 2018 · 3 comments · Fixed by #179
Assignees

Comments

@Decopi
Copy link

Decopi commented May 14, 2018

Hi @cowlicks !

It is very nice to see PrivacyPossum growing, with almost 600 users at Firefox. Congrats!

I have a simple request, without urgency/priority: Please, consider the possibility to change the PP' icon black color, with another color, blue, green, yellow, orange etc, which will contrast on browser black themes.

Thanks a lot!!

PS: The PP' counter color is "ok". No changes needed.

@cowlicks
Copy link
Owner

It would probably be easier, and look better, to just give the possum icon a background color instead of having it be transparent like it currently is.

@cowlicks cowlicks changed the title Simple request: Change black icon for browser black themes make browser icon more visible with dark browser themes May 15, 2018
@CollinChaffin
Copy link

Not intending to annoy but this has been open for a while and I hadn't seen a post to show the team the extent to which the PP icon really is totally invisible in a dark theme. Here is a screenshot of it next to the uBlock icon if it helps (I think it actually was even auto-adjusted brighter in the screenshot app as it appears even more invisible in reality but you get the idea):

snag_11-4-2018_17-42-53a

Thanks for providing such an awesome extension!

@cowlicks
Copy link
Owner

cowlicks commented Jul 7, 2019

@CollinChaffin fixed on master. I'll be making a release as soon as possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants