https://bugs.kde.org/show_bug.cgi?id=442539

--- Comment #19 from Nate Graham <n...@kde.org> ---
Git commit ab234fa25dfef4181a48a704e3b0d9c62aa80185 by Nate Graham, on behalf
of Vlad Zahorodnii.
Committed on 23/09/2021 at 13:54.
Pushed by ngraham into branch 'Plasma/5.23'.

wayland: Fix cross cursor in Xwayland apps

Startup code in plasmashell was changed so xsetroot is not called
anymore, which is sort of fine.

Unfortunately (or not?), it exposed a bug in kwin. Cursor::x11Cursor()
only works in the standalone X11 session.

On Wayland, Cursor::x11Cursor() will return XCB_NONE which results in
seeing cross cursor when there should be arrow cursor.

This change moves xcb_cursor_t look up code from X11Cursor to the base
Cursor class. In hindsight, I would like to introduce a window manager
class where the xcb cursor and other x11 specific code can be moved in
the future for better encapsulation of platform-specific code.


(cherry picked from commit ca1c72dd16699a5b8250e235f27e67f0f78adae9)

M  +2    -0    autotests/CMakeLists.txt
M  +1    -0    src/CMakeLists.txt
M  +35   -14   src/cursor.cpp
M  +1    -12   src/cursor.h
M  +1    -1    src/plugins/platforms/x11/standalone/CMakeLists.txt
M  +0    -44   src/plugins/platforms/x11/standalone/x11cursor.cpp
M  +0    -4    src/plugins/platforms/x11/standalone/x11cursor.h

https://invent.kde.org/plasma/kwin/commit/ab234fa25dfef4181a48a704e3b0d9c62aa80185

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to