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