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

Martin Gräßlin <mgraess...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |http://commits.kde.org/kwin
                   |                            |/3bc608939444f88dd855130982
                   |                            |ce176d393d3ab5
         Resolution|---                         |FIXED
             Status|CONFIRMED                   |RESOLVED
   Version Fixed In|                            |5.8.1

--- Comment #11 from Martin Gräßlin <mgraess...@kde.org> ---
Git commit 3bc608939444f88dd855130982ce176d393d3ab5 by Martin Gräßlin.
Committed on 06/10/2016 at 11:26.
Pushed by graesslin into branch 'Plasma/5.8'.

Only repeat one key

Summary:
So far KWin tried to repeat all pressed keys which should repeat. But
this is not how X11 and e.g. QtWayland handle it. There only one key -
the last one which got pressed - repeats. And this makes sense as the
key is used to generate a keysym and that one KWin caches. Thus the
logic so far resulted in incorrect keysyms to be generated during the
repeat. E.g. pressing a, pressing b, releasing b would repeat b instead
of the hold a as b was the last generated keysym.

This change addresses this problem and let's only one key repeat at a
time. When the currently repeating key gets released the repeat timer is
stopped and other hold keys won't repeat any more. This also matches the
behavior of X11 and QtWayland.
FIXED-IN: 5.8.1

Reviewers: #kwin, #plasma_on_wayland

Subscribers: plasma-devel, kwin

Tags: #plasma_on_wayland, #kwin

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

M  +0    -1    autotests/integration/globalshortcuts_test.cpp
M  +19   -23   keyboard_input.cpp
M  +5    -1    keyboard_input.h

http://commits.kde.org/kwin/3bc608939444f88dd855130982ce176d393d3ab5

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

Reply via email to