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
slack: Fix dark theme #65156
slack: Fix dark theme #65156
Conversation
Tagging people who reviewed original PR that caused the issue @NeQuissimus @joachifm Thanks! |
Caused by NixOS#61328
469ef0b
to
909d2c4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally, I can confirm it fixes slack-dark
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this, it fixes the theme.
- reviewed the diff and commit messages
- made sure ofBorg evals
- run nix-review without any failures
- run and tested the binaries
Thanks for the reviews! |
Thank you! |
Tangentially related, has the thread view always been broken under dark theme? Text is light gray on white background :D |
I think it worked at some point. |
Updated the theme and threads are working again: 8f99ec0 |
Motivation for this change
slack-dark was broken by #61328
ajax calls are not allowed to local fs in electron, fs module has to be used.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)