Skip to content

firefox, zen-browser: add darkReader option#950

Draft
0xda157 wants to merge 1 commit intonix-community:masterfrom
0xda157:firefox-dark-reader
Draft

firefox, zen-browser: add darkReader option#950
0xda157 wants to merge 1 commit intonix-community:masterfrom
0xda157:firefox-dark-reader

Conversation

@0xda157
Copy link
Copy Markdown
Contributor

@0xda157 0xda157 commented Mar 3, 2025

Haven't tested locally yet
Currently not working because dark reader doesn't have support for config via hm.

resolves #495
cc: @brckd

@0xda157 0xda157 force-pushed the firefox-dark-reader branch 4 times, most recently from 4df1861 to fb288c6 Compare March 3, 2025 04:29
@brckd
Copy link
Copy Markdown
Contributor

brckd commented Mar 3, 2025

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.

@voronind-com
Copy link
Copy Markdown

voronind-com commented Mar 3, 2025

I think @voronind-com made a Nix config that uses that.

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.

@brckd
Copy link
Copy Markdown
Contributor

brckd commented Mar 3, 2025

No issues apart from the fact that I need to allow the installation of unsigned ff extensions.

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.

I don't update the colors on a running ff instance as the browser itself requires a restart to apply new colors.

Shouldn't be a problem since that's how most apps behave.

Would be happy to help.

I appreciate it!

@voronind-com
Copy link
Copy Markdown

I wasn't able to successfully deploy these.

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 appreciate it!

I said that in a hope you could point me to the stuff I could do :p

@brckd
Copy link
Copy Markdown
Contributor

brckd commented Mar 3, 2025

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!

@0xda157
Copy link
Copy Markdown
Contributor Author

0xda157 commented Mar 4, 2025

Bummer that dark reader can't be configured with hm, I'll just leave this pr as a draft until dark reader adds support.

@trueNAHO
Copy link
Copy Markdown
Member

trueNAHO commented Mar 6, 2025

This PR would be able to resolve #253 and #495, right?

@0xda157
Copy link
Copy Markdown
Contributor Author

0xda157 commented Mar 7, 2025

This PR would be able to resolve #253 and #495, right?

It's won't resolve #253 because it's overriding the website's styling instead of suggesting a polarity.

@danth danth added the kind: feature Feature request label Mar 11, 2025
@0xda157 0xda157 force-pushed the firefox-dark-reader branch from fb288c6 to c2b83f7 Compare April 20, 2025 19:08
@trueNAHO
Copy link
Copy Markdown
Member

What is the state of this PR?

@0xda157
Copy link
Copy Markdown
Contributor Author

0xda157 commented Apr 29, 2025

What is the state of this PR?

waiting for darkreader/darkreader#12920 to be merged

@Letgamer
Copy link
Copy Markdown

Letgamer commented Mar 3, 2026

BryceBeagle/nixos-config#333
This is already possible without the upstream PR! @0xda157

@0xda157 0xda157 force-pushed the firefox-dark-reader branch from c2b83f7 to bf425c7 Compare April 8, 2026 21:48
@0xda157 0xda157 changed the title firefox: add darkReader option firefox, zen-browser: add darkReader option Apr 8, 2026
@stylix-automation stylix-automation bot added topic: home-manager Home Manager target topic: testbed Testbed changes topic: modules /modules/ subsystem labels Apr 8, 2026
@0xda157
Copy link
Copy Markdown
Contributor Author

0xda157 commented Apr 8, 2026

doesn't seem to be working when testing in testbed, would appreciated help debugging

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

Labels

kind: feature Feature request status: merge conflict Merge conflict topic: home-manager Home Manager target topic: modules /modules/ subsystem topic: testbed Testbed changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

idea: support a cross browser dark mode extension like dark-reader

6 participants