[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