-
Notifications
You must be signed in to change notification settings - Fork 35
Set icon theme and button layout for GNOME only if requested #116
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
base: master
Are you sure you want to change the base?
Conversation
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.
These shouldn't be gated by the global theme setting. This is needed globally or else window controls and icons won't match. In fact, on a first login, if the icon theme is not set to match the desktop, there will be missing icons in a lot of applications. You'll also see that GTK apps will be missing minimize and maximize buttons even though its the default for COSMIC.
|
For the default settings, it's possible to set session-dependent defaults just for the COSMIC session by creating a GSettings override file, e.g. In this way, the default settings will match with COSMIC, but still allows the user to set a custom button layout and icon theme for GTK apps. |
0b85c91 to
2672542
Compare
|
I updated this pull request to include a GSettings override file to match the default settings with COSMIC, so there will be no difference on first login. |
|
The button layout for window controls and icons should still be kept in sync with modifications made in COSMIC Settings. If you disable the minimize or maximize buttons in COSMIC Settings, that should also be applied to GTK applications. Same for icon themes. The global theme variable is specifically about overriding the GTK4/libadwaita CSS file. |
|
Why does the global theme variable apply only for setting Without this, it's impossible to set custom button layout for GTK apps, because it always gets overridden on login. |
|
Also, the Maybe it would better to implement these properties within the Settings portal in xdg-desktop-portal-cosmic. GTK no longer reads GSettings, and relies only on portal: https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/8923 |
This ensures that the icon theme and button layout for GNOME applications are not overwritten at login and when changed in the settings if the 'apply_theme_global' setting is disabled. This allows users to decide whether they want COSMIC to modify GNOME settings.
2672542 to
cab3a19
Compare
|
I opened another pull request: #117
Both solutions would be fine for me. |
This ensures that the icon theme and button layout for GNOME applications are not overwritten at login or when changed in the settings if the
apply_theme_globalsetting is disabled. This allows users to decide whether they want COSMIC to modify GNOME settings.