-
-
Notifications
You must be signed in to change notification settings - Fork 35
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
Extension settings unreadable using Firefox dark theme #71
Comments
Thanks for pointing this out, will look into it. |
Hi @glenneroo, @sjuxax. I was not able to reproduce this on my system, but I've pushed a style file (webext-base.css) that should fix this. Please do update! |
I pushed the update to Mozilla Add-ons, and Firefox should ideally silently update your extension. The latest version is 20.4.4, is that the one you have installed? |
I'm on 20.4.4 and it's still unfortunately the same. Did you set your FireFox to dark theme? Otherwise I guess it's one of my many extensions. |
That's unfortunate :( I still have not been able to reproduce this on my machine -- I'm using Firefox 74.0.1 on Linux Ubuntu 18.04. What's your system info? |
Half the battle of fixing bugs is being able to reproduce them ;) Firefox 74.0.1 on Windows 10. Here's a list of my extensions: |
Not fixed here either on Firefox 75b12 or Firefox 76b1 on Win10, nor Firefox 75b12 on Ubuntu 18.10 (haven't tested 76b1). Backticks work though. 😉 🎉 If you don't beat me to it @plibither8, I'll try to debug sometime this week and see if I can see what's going on. I did check the extension debug view briefly and saw some path errors, but didn't do anything more than a superficial glance so they may be related to something else, like installing the temporary extension incorrectly. 😅 |
Thanks for the details @glennero! @sjuxax that would be great since I'm not able to reproduce on my machine. Hopefully you'll be able to work it out :D. Thanks! |
Hi @glenneroo, @sjuxax. I've merged #76 (thanks @lookfirst!) which should fix the issue. I've published the update on the marketplaces (v20.4.14) too. |
Still unfortunately happening on my end with 20.4.14 and after restarting my browser. |
I'll try to debug this on my Win10 Firefox tonight when I get time. Thanks for pointing it out. |
darn that it doesn't fix FF... .14 hasn't hit chromestore yet so i can't confirm it is fixed there... |
Chrome will take a week or two to update - they're review process takes time, especially during these days. |
That's insane... =) Is it possible to easily clone the repo and load it as a dev extension? |
I can send over the "dist/" directory zipped, and you can load it up on your machine. How's that? |
You can unzip this and proceed! :) |
Thanks! It is fixed in Chrome now. Silly me... loading the extension that way, removed my settings though. =( oh well! |
Ahh that's unfortunate :/ |
This issue is also related to a DarkReader firefox addon bug-report I just submitted: darkreader/darkreader#5512 (comment) After disabling Refined-Hacker-News, I can once again read code blocks normally, which are otherwise white text on white background, the same as this plugin. |
Looking at the stylesheet https://github.com/plibither8/refined-hacker-news/blob/main/src/popup.styl#L6 it's not surprising that it becomes unreadable, since the colors look to be mostly hardcoded to a light background there. Some potential options to fix might be:
|
As seen in screenshot:
![image](https://user-images.githubusercontent.com/1424173/73121191-e68ed580-3f77-11ea-93b9-eabe2959be48.png)
To enable dark theme, menu > customize > theme (very bottom) > dark theme.
The text was updated successfully, but these errors were encountered: