<table><tr><td style="">ngraham created this revision.<br />ngraham added a reviewer: Frameworks.<br />Restricted Application added a project: Frameworks.<br />ngraham requested review of this revision.
</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/D10314" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>FEATURE: 389880<br />
FIXED-IN: 5.44</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Shortcut works and toggles the aside preview.</p>

<p>However, in doing so, another pre-existing bug is triggered: the file dialog doesn't eat the keyboard input, so if the app that spawned it has something bound to F11, you'll erroneously get the "Ambiguous shortcut" message (even though the keystroke is/should be passed only to the file dialog, not the host app). Help solving this would be appreciated.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10314" rel="noreferrer">https://phabricator.kde.org/D10314</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/filewidgets/kdiroperator.cpp</div></div></div><br /><div><strong>To: </strong>ngraham, Frameworks<br /><strong>Cc: </strong>michaelh, ngraham<br /></div>