Differential D8155

Add ctrl+k to shortcuts list for Clear Scrollback and Reset
AbandonedPublic
Actions

Authored by ngraham on Oct 6 2017, 2:44 AM.

Details

Reviewers
hindenburg
Group Reviewers
Konsole
KDE Applications
Summary

Adds ctrl+k to the shortcuts list for "Clear Scrollback and Reset" command, which is really useful. The ctrl+k shortcut is what macOS Terminal uses (well, its command+k equivalent), so this makes life a tiny bit easier for terminal-using Mac converts by letting them re-use their muscle memory.

Test Plan

Tested in KDE Neon. With fresh settings, the new shortcut is present by default:

Diff Detail

Repository
R319 Konsole
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
ngraham created this revision.Oct 6 2017, 2:44 AM
Restricted Application added a project: Konsole. · View Herald TranscriptOct 6 2017, 2:44 AM
Restricted Application added a subscriber: Konsole. · View Herald Transcript
ngraham edited the summary of this revision. (Show Details)Oct 6 2017, 2:46 AM
ngraham edited the test plan for this revision. (Show Details)
ngraham added reviewers: Konsole, KDE Applications.
ngraham edited the summary of this revision. (Show Details)
hindenburg added a comment.Oct 7 2017, 2:39 PM

You'll notice that Konsole doesn't use any Ctrl+Key shortcuts - that's because it sends those keys to the program running. All/most of Konsole's shortcuts add SHIFT. (ie Ctrl+Shift+K)

ngraham added a comment.Oct 7 2017, 8:07 PM

I have noticed that, but ctrl+k isn't a signal caught by anything, is it?

hindenburg added a comment.Oct 7 2017, 10:09 PM

It could be - the user could run any shell or program in Konsole

ngraham abandoned this revision.Oct 17 2017, 8:46 PM

Revision Contents
Changeset List

PathPackages
Msrc/SessionController.cpp (2 lines)

Diff 20382

View Options

src/SessionController.cpp

Loading...