Review Request 118630: [KQuickControls] Consider Super_L and Super_R as modifiers
Commit Hook
null at kde.org
Tue Jun 10 14:31:53 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118630/#review59697
-----------------------------------------------------------
This review has been submitted with commit 7eaa20924b99e3fa92ec3aaddef3631e7400f3e8 by Sebastian Kügler to branch master.
- Commit Hook
On June 9, 2014, 10:04 p.m., Sebastian Kügler wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118630/
> -----------------------------------------------------------
>
> (Updated June 9, 2014, 10:04 p.m.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Repository: kdeclarative
>
>
> Description
> -------
>
> Consider Super_L and Super_R as modifiers
>
> Without this patch, I can't use the meta key to assign shortcuts, as
> Super_L and Super_R are not considered as modifiers, so when I press
> meta (Super_L on my system), the shortcut is immediately accepted,
> before I get the chance to press another key.
>
> This patch requires the fix in
> https://bugreports.qt-project.org/browse/QTBUG-38428
> to be applied. With both patches, the QtQuick widgets to assign
> shortcuts works for me also with the Meta key.
>
> Suggested by David Edmundson in review request at
> https://git.reviewboard.kde.org/r/118581/
>
> CCBUG:335316
>
>
> Diffs
> -----
>
> src/qmlcontrols/kquickcontrols/private/keysequencehelper.cpp 7bf67ecb68c97086c0d858ef7c7f774ae0560983
>
> Diff: https://git.reviewboard.kde.org/r/118630/diff/
>
>
> Testing
> -------
>
> Tested assigning Meta + L to digital clock plasmoid, works with the patch, doesn't work without (displays garbage like in the linked bugreport).
>
>
> Thanks,
>
> Sebastian Kügler
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140610/392038b4/attachment.html>
More information about the Kde-frameworks-devel
mailing list