-
Notifications
You must be signed in to change notification settings - Fork 129
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
Support org.freedesktop.appearance API for dark/light theme switching #1313
Comments
This is already implemented. Relevant code can be found in src/conf.c:286 #1275 is a strange issue, but not caused by not implementing the FreeDesktop portal support. |
I also experience issue #1275 . Okay, seeing the code, doesn't look like it consults with portal and d-bus to fetch theme values? to reproduce
|
@RushingAlien I was maybe naivly expecting both interfaces GDBUS endpoint and GSettings schema to report the same setting value. If both places report different values how is the app to behave? About reproducing: can you elaborate a bit more? Do you run in Flatpak or is the effect visible without Flatpak too? |
Hello again, Sorry for the late reply. I don't think portals has a GSettings schema endpoint at all And to answer your question:
I have reproduced issue in both flatpak and non-flatpak I'm guessing you are using Ubuntu? Ubuntu downstreams a lot hacks to make things work |
This confirms my suspicion
I remove the conditional for the fdo codepath in liferea I can't see anything in What we have however is Of course, thiss means KDE has a different trigger for fdo settings |
Given the implementation by libadwaita (https://gitlab.gnome.org/GNOME/libadwaita/-/blob/main/src/adw-settings-impl-portal.c) we should entirely switch to DBUS only too. It is easier to maintain. |
Hello, GNOME, KDE, and other DEs have adopted a standard, desktop agnostic API for deciding themes, GNOME even goes so far as to make their dark settings that's not in gnome-tweaks only set dark/light this way.
Both of these toggle set dark theme only via the org.freedesktop.appearance API, hence issue #1275, the API used in gnome-tweaks is not touched through these toggles and is considered unsupported usecase
The API:
https://flatpak.github.io/xdg-desktop-portal/docs/#gdbus-org.freedesktop.portal.Settings
And don't worry, the API works outside of Flatpak
Apps that have implemented this API
The text was updated successfully, but these errors were encountered: