Review Request 124954: Add support for modifier only shortcuts on Wayland
Thomas Pfeiffer
colomar at autistici.org
Thu Aug 27 19:04:27 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124954/#review84486
-----------------------------------------------------------
Since this is indeed a very often required feature, why do we keep it hidden? If it can have negative side-effects, it we should warn users about them, not hide the whole feature from them.
- Thomas Pfeiffer
On Aug. 27, 2015, 4:04 p.m., Martin Gräßlin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124954/
> -----------------------------------------------------------
>
> (Updated Aug. 27, 2015, 4:04 p.m.)
>
>
> Review request for kwin, Plasma and Hans Chen.
>
>
> Repository: kwin
>
>
> Description
> -------
>
> On popular demand!
>
> This change tracks how modifiers are used and detects a modifier only
> key press/release. That is:
> * no other key is pressed when the modifier gets pressed
> * no other key gets pressed before the modifier gets released
>
> If such a press/release is detected, we call a configurable dbus call.
> The possible shortcuts can be configured in kwinrc, group
> "ModifierOnlyShortcuts". The following keys are supported:
> * Shift
> * Control
> * Alt
> * Meta
>
> As value it takes a QStringList (comma seperated string) with
> service,path,interface,method,additionalargs
>
> E.g. to invoke Desktop Grid effect on Meta key:
>
> [ModifierOnlyShortcuts]
> Meta=org.kde.kglobalaccel,/component/kwin/,org.kde.kglobalaccel.Component,invokeShortcut,ShowDesktopGrid
>
> I do not intend to add a config interface for it. Let's keep it a hidden
> way.
>
>
> Diffs
> -----
>
> input.h cfb693dc06a18ea9ca7cffb99b9a1318ee443e3a
> input.cpp 92724d7b7559dd460a8f5fbe17deb3c72024eed6
> options.h 07c5193e3bd205c5c8c22a305f4c1d87e16d175f
> options.cpp 64269d64bc49640bf2e4e925ce1969f2a5d6b96b
>
> Diff: https://git.reviewboard.kde.org/r/124954/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gräßlin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150827/a73491f1/attachment.html>
More information about the Plasma-devel
mailing list