Set mouse cursor theme and size

lxqt / lxqt-qtplugin

LXQt Qt platform integration plugin

https://lxqt.github.io

GNU Lesser General Public License v2.1

24 stars 14 forks source link

Set mouse cursor theme and size #85

Closed tsujan closed 2 months ago

tsujan commented 3 months ago

MouseCursorTheme and MouseCursorSize were introduced into Qt 6.5.

Closes https://github.com/lxqt/lxqt-qtplugin/issues/78

q234rty commented 3 months ago

I don't use a full lxqt session myself, but this patch seems to require cursor_size=@Size(48 48) instead of cursor_size=48 in ~/.config/lxqt/session.conf?

tsujan commented 3 months ago

@q234rty Thanks! Fixed now.

For some reason, I can't test it with my settings under LabWC. Does it make any difference to you?

q234rty commented 3 months ago

If the compositor implements wp-cursor-shape-v1 qt prefers using that, you can test with QT_WAYLAND_DISABLED_INTERFACES=wp_cursor_shape_manager_v1.

tsujan commented 3 months ago

you can test with QT_WAYLAND_DISABLED_INTERFACES=wp_cursor_shape_manager_v1.

Thanks for the tip!

LabWC doesn't implement it, but KWin does. I tested inside a nested kwin_wayland session and Qt6 apps had the correct cursor size with the patch and QT_WAYLAND_DISABLED_INTERFACES=wp_cursor_shape_manager_v1.

q234rty commented 3 months ago

Works for me also, with hyprland and QT_WAYLAND_DISABLED_INTERFACES=wp_cursor_shape_manager_v1. Please give some love to my other issue as well :)

tsujan commented 3 months ago

Please give some love to my other issue as well

We need the opinions of other devs about it. I'm calling them on the report page....

tsujan commented 3 months ago

Any objection to merging this?