<table><tr><td style="">ngraham added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D7354" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Good point. It looks like setDefaultShortcut() can take a QList of shortcuts instead of just a single one, which will add the new shortcut without blowing away the current one. This works since Konsole currently has no secondary shortcut, so we wouldn't be breaking anybody's secondary shortcut muscle memory.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7354" rel="noreferrer">https://phabricator.kde.org/D7354</a></div></div><br /><div><strong>To: </strong>ngraham, KDE Applications, Konsole<br /><strong>Cc: </strong>aacid, Konsole, hindenburg<br /></div>