Bug 383603

Summary: [PATCH] Konsole should adopt standard tab switching shortcuts in KStandardShortcut::tabPrev() and KStandardShortcut::tabNext()
Product: [Applications] konsole Reporter: Nate Graham <nate>
Component: tabbarAssignee: Konsole Developer <konsole-devel>
Status: RESOLVED FIXED    
Severity: wishlist CC: simonandric5
Priority: NOR    
Version: master   
Target Milestone: ---   
Platform: Other   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=80657
Latest Commit: Version Fixed In: 17.12
Sentry Crash Report:

Description Nate Graham 2017-08-16 17:26:53 UTC
The KDE standard keyboard shortcut for switching tabs was changed to ctrl+pgup/pgdn in https://phabricator.kde.org/D6553. This new shortcut works in Konsole, because ctrl+pgup/pgdn don't send any special shell signals or anything. Therefore, Konsole should adopt the same shortcuts to be consistent with other KDE software (and most software that uses tabs, these days).

This will require a change to ViewManager.cpp, either by changing the shortcut keys, or preferably by adopting KStandardShortcut::tabPrev() and KStandardShortcut::tabNext().
Comment 1 Nate Graham 2017-08-16 17:42:16 UTC
I have created a quick and dirty patch/phabricator diff available at https://phabricator.kde.org/D7354
Comment 2 Albert Astals Cid 2017-08-27 23:01:50 UTC
Git commit 374b602d52447caab9f934e781ae170bd994da7a by Albert Astals Cid.
Committed on 27/08/2017 at 23:01.
Pushed by aacid into branch 'master'.

Add new standard KDE tab switching shortcuts (ctrl+pgup/pgdn)

Summary:

Add the new KDE standard tab switching shortcuts that were introduced in https://phabricator.kde.org/R237:d85a57645f94d2d087711d7c608a0c5a46ed7ede.

Test Plan: Built and tested with current git master in up-to-date KDE Neon. Expected behavior is seen (default shortcuts have become shift+left/right and cmd+pgup/pgdn).

Reviewers: #kde_applications, #konsole, hindenburg, aacid, ltoscano, pino, dfaure

Reviewed By: dfaure

Subscribers: dfaure, davidedmundson, aacid, #konsole

Tags: #konsole

Differential Revision: https://phabricator.kde.org/D7354

M  +4    -2    src/ViewManager.cpp

https://commits.kde.org/konsole/374b602d52447caab9f934e781ae170bd994da7a