firefox, zen-browser: add darkReader option#950
firefox, zen-browser: add darkReader option#9500xda157 wants to merge 1 commit intonix-community:masterfrom
Conversation
4df1861 to
fb288c6
Compare
|
Good idea! Last time I looked though, dark reader couldn't be configured using extension storage or policies, but please correct me if I'm wrong. In that case, we could check out darkreader/darkreader#12920 which aims to add support for policies. I think @voronind-com made a Nix config that uses that. darkreader/darkreader#2168 seems to offer a solution for syncing colors during runtime alongside this nix config. |
Yes, I do indeed patch the Dark Reader with that PR and use colors from the Stylix. No issues apart from the fact that I need to allow the installation of unsigned ff extensions. I don't update the colors on a running ff instance as the browser itself requires a restart to apply new colors. Would be happy to help. |
That is indeed a problem because AFAIK this cannot be automated and is a security regression. The DarkReader README also mentions that signing keys for Firefox are available, but I wasn't able to successfully deploy these.
Shouldn't be a problem since that's how most apps behave.
I appreciate it! |
Me neither, so here we are. The best solution would be to ask the dev to merge the pull request, maybe from the Stylix team. He should be aware about that PR as he left some comments there before.
I said that in a hope you could point me to the stuff I could do :p |
|
I think it's not worth it to create a solution just now. I'd rather wait for the darkreader PR to be merged and assist there. But hey, this isn't my PR! |
|
Bummer that dark reader can't be configured with hm, I'll just leave this pr as a draft until dark reader adds support. |
fb288c6 to
c2b83f7
Compare
|
What is the state of this PR? |
waiting for darkreader/darkreader#12920 to be merged |
|
BryceBeagle/nixos-config#333 |
c2b83f7 to
bf425c7
Compare
|
doesn't seem to be working when testing in testbed, would appreciated help debugging |
Haven't tested locally yetCurrently not working because dark reader doesn't have support for config via hm.
resolves #495
cc: @brckd