Set icon theme and button layout for GNOME only if requested#116
Set icon theme and button layout for GNOME only if requested#116City-busz wants to merge 1 commit intopop-os:masterfrom
Conversation
There was a problem hiding this comment.
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.