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/plasma-devel/attachments/20140610/392038b4/attachment.html>


More information about the Plasma-devel mailing list