<table><tr><td style="">ngraham accepted this revision.<br />ngraham added a comment.<br />This revision is now accepted and ready to land.
</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/D14201">View Revision</a></tr></table><br /><div><div><p>Yes, I like it the idea. The chosen shortcut makes sense, and I'm always in favor of standardizing these across apps. Let's hold off on landing this patch (if accepted by everyone) until  Cirkuit has had a release with <a href="https://phabricator.kde.org/D14202" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D14202</a> so we don't wind up with the dreaded "Ambiguous Shortcut!" dialog.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R237 KConfig</div></div></div><br /><div><strong>BRANCH</strong><div><div>shortcut-actual-size (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14201">https://phabricator.kde.org/D14201</a></div></div><br /><div><strong>To: </strong>muhlenpfordt, dfaure, broulik, ngraham<br /><strong>Cc: </strong>rkflx, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>