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

Spot does not respect system theme #209

Closed
ghost opened this issue May 7, 2021 · 13 comments
Closed

Spot does not respect system theme #209

ghost opened this issue May 7, 2021 · 13 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists question Further information is requested

Comments

@ghost
Copy link

ghost commented May 7, 2021

Spot stays in Adwaita dark even though I use Adwaita light.

@ghost ghost added the bug Something isn't working label May 7, 2021
@xou816
Copy link
Owner

xou816 commented May 7, 2021

That's somewhat intended, but I'm open to changing the defaults if that's too weird of a choice!

See #187

@xou816 xou816 added the duplicate This issue or pull request already exists label May 7, 2021
@ghost
Copy link
Author

ghost commented May 7, 2021

Thank you for your quick response! It is not weird but it is not the expected behavior.

@schlauerlauer
Copy link

I think the dark theme by default is fine,
but when I change the system theme to Adwaita-dark Spot switches to the light theme 😄
and I can't change it to dark again with the keyfile

So Spot uses the opposite to the system theme?
(only a nitpick for me though)

@xou816
Copy link
Owner

xou816 commented May 20, 2021

Ahah that's... weird! I can't reproduce the issue on my end, if I use Adwaita, it will use the light or dark variant based on my gsettings, and if I use Adwaita-dark, it's always dark 🤔

I suppose you're using the flatpak version? Is there anything special about your install? (modified Adwaita theme, something else...?)

@schlauerlauer
Copy link

Okay I tried it again and now it respects the Adwaita theme (but changing the setting in the keyfile does nothing).
Weird.. I rebooted maybe that helped 🙂

Yes it's the flatpak version
and no there's nothing special about the Adwaite theme.

@lobau
Copy link

lobau commented Jun 11, 2021

I have the same issue. I use Adwaita Light but Spot is in dark. If I switch the system in Dark mode, Spot becomes Light. I wish Spot would just follow the system theme :-)
Screenshot from 2021-06-10 18-58-29

@xou816
Copy link
Owner

xou816 commented Jun 11, 2021

Definitely weird ahah, are you using the same distro per chance? Do you have the same issue with other Flatpak apps? (one that allows switching to the dark theme preferably).

I am not doing anything special with the theme (just exposing a gsetting to prefer the dark variant).

Things that could be happening I think:

  • Adwaita-dark acts as a theme for which the dark variant is the light version of Adwaita (unlikely);
  • Adwaita-dark is seen as a theme fully distinct from Adwaita, and if it has no counterpart on the flatpak side of things, flatpak defaults to using Adwaita (but no longer respects the setting to prefer the dark variant for some reason).

If the latter is correct, it might be worth checking if you have Adwaita-dark installed on your flatpak install:
$ flatpak list | grep theme
Adwaita dark GTK theme org.gtk.Gtk3theme.Adwaita-dark 3.22 flathub user

@lobau
Copy link

lobau commented Jun 11, 2021

I'm using PureOS (GNOME 3.38.4) with the default Adwaita theme.
I have Adwaita Dark installed. Spot looks great in dark mode on my system :)
The problem is that to be in dark mode my whole system has to be in light mode.

Screenshot from 2021-06-11 11-12-14

Here is the output of the command:
$ flatpak list | grep theme Adwaita theme org.kde.KStyle.Adwaita 5.15 flathub system

I tried to find another app that enforce a dark theme by defaut to see if other apps behave like this but I cannot find one. Do you know of another gtk app that enforce a dark theme?

@ondras12345
Copy link
Contributor

Do you know of another gtk app that enforce a dark theme?

Shortwave

@xou816
Copy link
Owner

xou816 commented Jun 11, 2021

Thanks for the details! Looks like Adwaita-dark is not installed on the flatpak side for your system. Try installing it and see if it helps :)

@ghost
Copy link
Author

ghost commented Jun 12, 2021

@xou816 do you have any plans to release it as apt package?

@lobau
Copy link

lobau commented Jun 13, 2021

Thanks for the details! Looks like Adwaita-dark is not installed on the flatpak side for your system. Try installing it and see if it helps :)

It might help Spot looks Dark while the rest of the system is dark too, but I'm not sure how that would help the original request of this bug, which is to respect the system theme?
I would like to use Spot in light mode, as the rest of my system is in light mode, which I believe is the request of the OP too :)

@xou816
Copy link
Owner

xou816 commented Jun 13, 2021

There are two things at play here, which is why it gets confusing, let me summarize (I should probably add something to the README, it's not the first time this topic is brought up):

  • Spot exposes a gsetting to prefer the dark variant of your Gtk theme (similarly to other Gtk applications, eg Lollypop); the default value is "true" which is why when everything else works correctly theme-wise, Spot uses the dark theme (see Spot uses wrong Flatpak-GTK Theme #187 (comment) );

  • for flatpak apps to use a custom Gtk theme (that includes Adwaita-dark!), the theme has to be installed on the flatpak side as well (org.gtk.Gtk3theme.Adwaita-dark in that case)

Hope this makes things clearer :)

@xou816 xou816 added the question Further information is requested label Jun 13, 2021
@ghost ghost closed this as completed Jun 13, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants