Skip to content
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

Dark theme doesn't work when using Mullvad browser (based on tor browser) with security slider at anything other than "standard" #50

Closed
AmyMoriyama opened this issue May 22, 2023 · 6 comments · Fixed by #87
Labels

Comments

@AmyMoriyama
Copy link

Describe the bug
A clear and concise description of what the bug is.
When using the Mullvad browser (basically tor browser without the tor stuff, so this probably applies to tor browser too), the default theme is always light even when set to dark if the security slider is set to anything other than "standard".

Steps to reproduce
Steps to reproduce the behavior:

  1. Get Mullvad browser: https://mullvad.net/en/download/browser/linux
  2. Extract, execute
  3. Set security slider to anything other than standard
  4. MY EYES o_o

Expected behavior
I would think the security slider would have no effect on whether a site theme is dark or light.

Screenshots
If applicable, add screenshots to help explain your problem.
n/a

Device information (please complete the following information):
cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux 11 (bullseye)"
NAME="Debian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

Mullvad browser: 12.0.6 (based on Mozilla Firefox 102.11.0esr) (64-bit)

Additional context
Add any other context about the problem here.
Trying to get a simple and fast website that is dark and doesn't need javascript. Which reminds me, seems javascript still is included in the public website even when configured not to be used. Of course, I am new to this whole hugo thing so could be user error.

@hugo-sid
Copy link
Owner

hugo-sid commented May 25, 2023

Thanks for reporting this bug @AmyMoriyama.

I guess when the security slider is set to anything other than "standard", scripts are blocked. Can you confirm this?

Trying to get a simple and fast website that is dark and doesn't need JavaScript. Which reminds me, seems JavaScript still is included in the public website even when configured not to be used.

This theme uses JavaScript to implement several features.

@kusyka911
Copy link
Contributor

kusyka911 commented May 26, 2023

As far as I know privacy oriented browsers (brave, mullvad etc.) prevent detecting system-level dark mode. And in case JS is disabled in your browser you will not be able to switch theme manually. Probably there is ability to enable dark-mode detection on browser flags page (not settings) chrome:https://flags for chrome based or about:config for firefox based. Or you can set dark mode by default in theme configuration for your website.

As an option we can add configuration param to hide theme switching button. So you can configure your website to hide theme switcher and use theme in dark only mode.

@AmyMoriyama
Copy link
Author

Or you can set dark mode by default in theme configuration for your website.

I tried that. Either I changed the wrong thing or it still defaults to light theme even when set to dark. And yes, the website is meant to be loaded without javascript needed.

@github-actions
Copy link

github-actions bot commented Jun 5, 2023

This issue has been marked stale due to inactivity. It will auto-close in 7 days without an update.

@github-actions github-actions bot added the Stale label Jun 5, 2023
@github-actions
Copy link

This issue was auto-closed due to inactivity.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jun 13, 2023
@kusyka911 kusyka911 mentioned this issue Jul 18, 2023
5 tasks
@kusyka911
Copy link
Contributor

kusyka911 commented Jul 18, 2023

Or you can set dark mode by default in theme configuration for your website.

@AmyMoriyama
I'm sorry for long response time. This wasn't working correctly. Even when you set default color to dark, it's still using theme detection in no-script browsers. I've made a PR to fix this. #85

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants