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

White theme in a dark theme #30766

Closed
Kitanit opened this issue Jun 2, 2023 · 43 comments · Fixed by brave/brave-core#18922
Closed

White theme in a dark theme #30766

Kitanit opened this issue Jun 2, 2023 · 43 comments · Fixed by brave/brave-core#18922
Assignees
Labels
feature/themes needs-more-info The report requires more detail before we can decide what to do with this issue. OS/Desktop OS/Linux priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA/Yes release-notes/include

Comments

@Kitanit
Copy link

Kitanit commented Jun 2, 2023

On Linux Mint 21.1 Cinnamon, after the latest stable update (Brave: 1.52.117 Chromium: 114.0.5735.90), when I reopen the browser if the dark theme is turned on, the white one is turned on even if the dark one is in the settings, it is fixed by turning on the white one and turning back the dark one

simplescreenrecorder-2023-06-02_10.23.19.mp4
@kraizelburg
Copy link

This is only temporary until you close and open the browser again. It is a known bug in latest linux version.

@pedrof04
Copy link

pedrof04 commented Jun 2, 2023

Same problem here on Linux Mint 21.1 Mate.

@salvaft
Copy link

salvaft commented Jun 2, 2023

Same problem. Linux Mint 21.1 xfce

This is only temporary until you close and open the browser again. It is a known bug in latest linux version.
After I reopen the browser the light version is set again...

@zquestz
Copy link

zquestz commented Jun 2, 2023

Confirmed on Arch with Cinnamon. Would be great to see this fixed.

@Misza13
Copy link

Misza13 commented Jun 3, 2023

Same here on Manjaro, package brave-browser 1.52.117-2.
Switching to light then dark fixes it until the next restart of the browser.

When I use GTK theme instead of classic, then most of the UI stays dark, but icons on the new tab page have white backgrounds after restart and some pages (like YouTube) load in light mode. Again, switch to light, then dark fixes until restart.

@Pieter-Visscher
Copy link

issue is also present on Debian 12.

@solo-dante
Copy link

Same problem here on Linux Mint 21.1 Mate.

+1

@charanravi-online
Copy link

Issue is reproduced on Ubuntu 22.04.2 LTS as well.

@DustyGanteng
Copy link

Same problem here on Linux Mint 21.1 Mate.

+1
workaround: gsettings set org.gnome.desktop.interface color-scheme prefer-dark

@solo-dante
Copy link

gsettings set org.gnome.desktop.interface color-scheme prefer-dark

Thanks @DustyGanteng that worked.

@Terraminator
Copy link

Terraminator commented Jun 4, 2023

same thing for me. Thank you @DustyGanteng. worked for me! (Linux Mint 21.1 Vera)

@pedrof04
Copy link

pedrof04 commented Jun 4, 2023

Thank you very much @DustyGanteng , worked on Linux Mint 21.1 Mate.

@AuracleTech
Copy link

AuracleTech commented Jun 5, 2023

image

Linux Mint XFCE 21.1 64bit x86_64

EDIT : gsettings set org.gnome.desktop.interface color-scheme prefer-dark someone else said fixed my issue (for now at least?)

@Brave-Matt
Copy link

@rebron @fallaciousreasoning can we have someone take a look at this? Likely an easy fix and good for quality of life for Linux users.

@rebron rebron added OS/Linux priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include labels Jun 5, 2023
@rebron
Copy link
Collaborator

rebron commented Jun 6, 2023

@Kitanit Are you seeing this in Chrome as well or is this specific to Brave? We want to see if this is an upstream issue.

@rebron rebron added the needs-more-info The report requires more detail before we can decide what to do with this issue. label Jun 6, 2023
@Kitanit
Copy link
Author

Kitanit commented Jun 6, 2023

@Kitanit Are you seeing this in Chrome as well or is this specific to Brave? We want to see if this is an upstream issue.

Sorry I don't use Chrome

@zsolaj
Copy link

zsolaj commented Jun 7, 2023

It also ignores the "Show tab search button" switch. It fixes itself once the setting gets enabled and disabled again, but only until the next browser start.
The issue started at the same time as the issue in the title, so I guess they're linked.
(Linux Mint 21.1 Cinnamon, kernel 5.15.0-73, Brave version 1.52.122 /did this on the previous stable release, too/)

@simonhong
Copy link
Member

simonhong commented Jun 15, 2023

Same on Fedora 38 with i3

Version 1.52.117 Chromium: 114.0.5735.90 (Official Build) (64-bit)

brave1

brave2

Thanks @Cakez0r . Did you set light theme as your os system theme?

@simonhong
Copy link
Member

Can I have some more help for testing?
I made test nightly build(brave-browser-nightly_1.54.61_amd64.deb and brave-browser-nightly-1.54.61-1.x86_64.rpm) with my draft PR.
Unfortunately, my machine doesnt' have this issue so can't verify this test build could fix or not.
It would be really great help if anyone can install this build and test whether this issue is resolved or not. Thanks all :)

@digitalextremist
Copy link

@simonhong I am testing this on a fresh install of Linux Mint 21.1 shortly.

@digitalextremist
Copy link

digitalextremist commented Jun 15, 2023

The issue is resolved in the nightly build you posted @simonhong. Nice work!

Confirmed fix is working on a fresh OS & package install manifesting the issue!

Any one else reproduce the success, or are there still failing cases outside Linux Mint 21.1?

@simonhong
Copy link
Member

The issue is resolved in the nightly build you posted @simonhong. Confirmed on a fresh install manifesting the issue!

Nice work! Any one else reproduce the success, or are there still failing cases outside Linux Mint 21.1?

@digitalextremist Thank you for all the help, it is much appreciated! 👍🏼

@digitalextremist
Copy link

It seems there is another confirmation with Linux 21.0 ( not 21.1 )

https://community.brave.com/t/dark-mode-resests-after-browser-is-closed/489864/50

@digitalextremist
Copy link

Seems there are two threads of testers with @simonhong here on GitHub but another glowing response, solving the issue on another Linux 21.1 system.

@simonhong
Copy link
Member

Seems there are two threads of testers with @simonhong here on GitHub but another glowing response, solving the issue on another Linux 21.1 system.

@digitalextremist Many thanks!!!

@Cakez0r
Copy link

Cakez0r commented Jun 16, 2023

Works for me also in your test build. Thank you!

@simonhong
Copy link
Member

Works for me also in your test build. Thank you!

@Cakez0r Thanks for your help! Current nightly channel includes this fix :)

@felipeabou
Copy link

I had this issue and your fix worked for me. I'm using Linux Mint 21.1.

Thank you very much, @simonhong !!!

@kjozwiak
Copy link
Member

The above requires 1.53.107 or higher for 1.53.x verification 👍

@LaurenWags
Copy link
Member

LaurenWags commented Jun 29, 2023

Verified with Ubuntu 20.04.6 LTS:

Brave	1.53.107 Chromium: 115.0.5790.40 (Official Build) (64-bit) 
Revision	071c9ddea889c3c7887daf4eac13fed72d4fff62-refs/branch-heads/5790@{#979}
OS	Linux

Reproduced issue as described in #30766 (comment):

Example Example
3 4

Verified that theming is working as expected on Linux using the STR/Cases outlined via brave/brave-core#18922 (comment) as per the following:

Ensured that Brave is always using the correct theming when switching the OS theme. Examples of cases:

  • ensured that Brave uses Dark theme when set even though the OS theme is set as Light
  • ensured that Brave uses Light theme when set even though the OS theme is set as Dark
  • ensured that restarting multiple times didn't cause Brave to use the incorrect theme.
Example Example
1 2

Also ensured that Use GTK was working correctly. When Use GTK, ensured that OS elements match the current theme but the theme shouldn't be changed within Brave as per brave/brave-core#18922 (comment).

Example Example
1 2
3 4

@TzourCode
Copy link

Same problem here on Linux Mint 21.1 Mate.

+1 workaround: gsettings set org.gnome.desktop.interface color-scheme prefer-dark

Thanks mate worked for me to!!! @DustyGanteng 💪

Linux Mint 21.1 Cinnamon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/themes needs-more-info The report requires more detail before we can decide what to do with this issue. OS/Desktop OS/Linux priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.