-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
appearance under wayland: fix customizing themes #757
Conversation
Fixes |
We got the same Travis issue, again affecting only Fedora. Debian build finished fine
|
@lukefromdc as I said we need to apply this mate-desktop/mate-desktop.org@6a6aa9f to every repo to fix this issue |
d610d7b
to
b29a2e8
Compare
#758 fixed travis builds in a test |
fc7ec90
to
54b4404
Compare
tested cursor scheme/size - works like a charm. |
Since wayland compositors such as wayfire follow the GNOME interface gsettings values, we need to set them here too
54b4404
to
649fad9
Compare
*Note that only compositors that read the cursor size from gsettings will apply this, wayfire does not at this time *This is also true of cursor theme: the compositor must read it from gsettings for it to be applied from the control center
I found the place where cursor size changes are applied and was able to change the GNOME gsettings value from there. Unfortunately, I also found that Wayfire does not use gsettings to set cursor theme or size at all unless this changed in some recent commit. This the the real reason I am not seeing any response to changing cursor theme or size via gsettngs.I can change it just fine from WCM (wayfire configuration manager) but it appears some other compositors do support changing the cursor size and theme from gsettings. At least some versions of sway have been reported to be able to read cursor size and theme from gsettings My vote is for adding a warning dialog to indicate that some compositors may ignore gsettings for cursor theme and size, and include cursor theme and size support in this. thoughts? |
Also see |
I use wayfire - git snapshot actually. But I did not set up any exotic plugins except of rounding corners over it. And - it works for me. How - I do not know, maybe it is done by gtk through some wayland-wlroots call, but it works. And my wayfire is controlled by ini file - not by gsettings, Ok - writing this mail I got the point how it works. So - when mu cursor is hoverinhg over gtk window (or over Qt window ) - it pains cursor theme from org.gnome gsettings. When I hover over window decorations or Xorg(Xwayland) window it takes wayfire ini file settings. So it actually works excatly as Mate+Compiz in Xorg session. Gtk overrides the compositor settings. Or WM settings. So f you set up some fancy cursor in Wayfire.ini but have some legacy in gsettings it will behave inconsistently. And it is not a wayland-introduced bug (as I had same effect with compiz) - it is just how Mate works with external wm or compositor. And I do not see many solutions here. Well - solution 1 - is to make a Wayfire add-on like we had on compiz - "Mate-compatibilty" that will read gsettings cursor theme and force it over the one from ini file. or make add-on code for mate-settings - if wayfire detected (display = wayland + wayfire.ini is writeable ) - duplicate settings to wayfire.ini. Wayfire addon seems to be better way as it is going same ditrection as compiz was doing. Anyway cursor settings in org.gnome are needed because no matter how wayfire is acting now the org.gnome settings are respected. Over web-browsers, libreoffice or caja-painted desktop org.gnome cursor will be. |
Just confirmed that even though wayfire has its own ability to specify the cursor, the GNOME gsettings is in fact applied over a GTK window. Changing size works on my setup, the old hardcode I put somewhere for a decade ago to keep early QT4 apps from changing the cursor still means the theme change is visible only in dconf-editor. Anyway, we now have two tests confirming this part of the code works in and only in GTK apps. |
Does anyone else want to review this before merging? We have a successful test by the reporter of the original issue, and all the travis checks pass |
Since wayland compositors such as wayfire follow the GNOME interface gsettings values, we need to set them here too.
*Otherwise we can only reliably select existing metathemes or sometimes a saved custom theme.