Review Request: Fix meta+shift+tab and similar global shortcuts.

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101523/#review4245
-----------------------------------------------------------


This review has been submitted with commit 6026c3a32056bdbae6964e25abf712870edf5aac by Simon Persson to branch master.

- Commit


On June 6, 2011, 11:24 a.m., Simon Persson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101523/
> -----------------------------------------------------------
> 
> (Updated June 6, 2011, 11:24 a.m.)
> 
> 
> Review request for KDE Runtime and Michael Jansen.
> 
> 
> Summary
> -------
> 
> When entering (ctrl,alt,meta+) shift+tab as a new shortcut KKeySequenceWidget gets "backtab" as the key pressed but records "tab", because it looks/sounds better I guess... When either of backtab or tab is used with shift as a shortcut in an application Qt triggers the shortcut. kglobalaccel gets the keypress event as "backtab" which is correct but only triggers shortcuts with "backtab" in them and such shortcuts are not possible for the user to enter using kshortcutseditor.
> 
> This patch makes kglobalaccel look for both tab and backtab when triggered, same as Qt does.
> 
> KWin should possibly be patched to have alt+shift+tab as its default shortcut, currently it has alt+shift+backtab (which is why it works even without this patch...). That would solve the issue that currently a user can enter alt+shift+tab as shortcut for something else and the conflict is not detected.
> 
> 
> This addresses bugs 174142, 258467, 261296, and 274006.
>     http://bugs.kde.org/show_bug.cgi?id=174142
>     http://bugs.kde.org/show_bug.cgi?id=258467
>     http://bugs.kde.org/show_bug.cgi?id=261296
>     http://bugs.kde.org/show_bug.cgi?id=274006
> 
> 
> Diffs
> -----
> 
>   kglobalaccel/globalshortcutcontext.cpp fc2e6a6 
> 
> Diff: http://git.reviewboard.kde.org/r/101523/diff
> 
> 
> Testing
> -------
> 
> Tested using master branch, running in a Xephyr session. Switching between plasma activities with meta+shift+tab now works.
> 
> 
> Thanks,
> 
> Simon
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110629/945a547d/attachment.htm>