Review Request 124213: Add standard shortcut for "new tab" action
Simon Persson
simon.persson at mykolab.com
Tue Jun 30 09:13:31 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124213/
-----------------------------------------------------------
Review request for KDE Frameworks.
Repository: kconfig
Description
-------
QKeySequence already has AddTab as a standard shortcut, but it was missing in KStandardShortcut. Main problem with this is that users could not configure the shortcut globally. This is of course only a first step, applications would have to be updated to use this also.
Since QKeySequence uses Ctrl+Shift+N and Ctrl+T for this action when running on KDE, this patch just goes with that as the chosen default.
Also fixes documentation error for tabNext and tabPrev that I happened to notice.
Diffs
-----
src/gui/kstandardshortcut.h 5bb07fb
src/gui/kstandardshortcut.cpp 8400737
Diff: https://git.reviewboard.kde.org/r/124213/diff/
Testing
-------
Compiles, test suite ok.
Thanks,
Simon Persson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150630/134ad259/attachment.html>
More information about the Kde-frameworks-devel
mailing list