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

Martin Flöser <mgraess...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/kwi
                   |                            |n/a6dee74ee455d1da47dd5c9d5
                   |                            |5a84adbb5e1426a
         Resolution|---                         |FIXED
   Version Fixed In|                            |5.10.3
             Status|CONFIRMED                   |RESOLVED

--- Comment #53 from Martin Flöser <mgraess...@kde.org> ---
Git commit a6dee74ee455d1da47dd5c9d55a84adbb5e1426a by Martin Flöser.
Committed on 20/06/2017 at 04:56.
Pushed by graesslin into branch 'Plasma/5.10'.

Workaround Qt regression of no longer delivering events for the root window

Summary:
With qtbase 2b34aefcf02f09253473b096eb4faffd3e62b5f4 we do no longer get
events reported for the X11 root window. Our keyboard handling in effects
like PresentWindows and DesktopGrid relied on that.

This change works around the regression by calling winId() on
qApp->desktop() as suggested in the change. This is a short term solution
for the 5.10 branch.

This needs to be addressed properly by no longer relying on Qt in this
area. KWin already does not rely on Qt for Wayland in that area and is
able to compose the QKeyEvents. This should also be done on X11. It just
needs some more hook up code for xkb, but that's needed anyway to improve
modifier only shortcuts and friends.
FIXED-IN: 5.10.3

Reviewers: #kwin, #plasma

Subscribers: plasma-devel, kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D6258

M  +6    -0    effects.cpp

https://commits.kde.org/kwin/a6dee74ee455d1da47dd5c9d55a84adbb5e1426a

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

Reply via email to