[Differential] [Request, 163 lines] D4304: Split Keyboard Repeat handling into a dedicated InputEventSpy
Martin Gräßlin
noreply at phabricator.kde.org
Fri Jan 27 06:09:43 UTC 2017
graesslin created this revision.
graesslin added reviewers: KWin, Plasma on Wayland.
Restricted Application added subscribers: kwin, plasma-devel.
Restricted Application added projects: Plasma on Wayland, KWin.
REVISION SUMMARY
So far the keyboard repeat handling was triggered directly from
KeyboardInputRedirection::processKey. With the introduction of
InputEventSpies it is no longer required to be done like that, we can
split it out into a dedicated spy.
This means that processKey only has to care about processing the key
and allows us to better extend in future. So far keyboard repeat is
only functional for libinput based platforms. But it should also be
possible to use it in nested setups. By splitting it out we can
prepare for that.
TEST PLAN
Auto-test using repeat still passes
REPOSITORY
R108 KWin
BRANCH
keyboard-repeat
REVISION DETAIL
https://phabricator.kde.org/D4304
AFFECTED FILES
CMakeLists.txt
keyboard_input.cpp
keyboard_input.h
keyboard_repeat.cpp
keyboard_repeat.h
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, eliasp, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170127/c3f3c971/attachment.html>
More information about the Plasma-devel
mailing list