search

zaidka / cliget

A Firefox addon to download login-protected files from the command line
https://addons.mozilla.org/en-US/firefox/addon/cliget/
Mozilla Public License 2.0
357 stars 57 forks source link

cliget shows black text on dark background #59

Open j000 opened 6 years ago

j000 commented 6 years ago

Please set foreground colours everywhere you set background colours. I'm using dark desktop theme and Firefox follows it. Now I'm getting black text (default) on dark background (from css, default is light grey). screenshot_20180120_110119 screenshot_20180120_110139

zaidka commented 6 years ago

This looks like a bug in Firefox itself. Cliget uses browser styles and doesn't define any color or background color attributes in CSS.

Will see if we can somehow detect if a dark theme is being used and correct the styles accordingly.

j000 commented 6 years ago

If Cliget does not set any styles please close this issue :)

I have another extension with same problems, so it's definitely Firefox's fault.

zaidka commented 6 years ago

There might still be something we can do about it. I'm thinking the function getComputedStyle() can help here to detect if a dark theme is being used.

pinophyta commented 6 years ago

Maybe an option/checkbox in settings page for white bg? I use black bg for newtab/blank and having the same issue.