<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: Frameworks, Dolphin, markg, elvisangelaccio.<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/D10617" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>My patch <a href="https://phabricator.kde.org/D10314" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D10314</a> was insufficiently tested (sorry about that!) and introduced an "Ambiguous Shortcut" warning, because "show/hide inline preview" was already bound to F11.</p>

<p>This patch solves that issue by changing the shortcut to F12. F11 was never appropriate for it, since F11 was used for something different in Dolphin anyway. DOlphin doesn't currently have a shortcut for "show hide inline previews", so there's nothing to be consistent with. I used F12 in the file dialogs since it's unused there and in Dolphin, too, so we could add it there if we really wanted to.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Tested with Kate. F11 toggles the aside preview, and F12 toggles the inline preview, like they should. No more conflicts</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>solve-file-dialog-ambiguous-shortcut-issue (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10617" rel="noreferrer">https://phabricator.kde.org/D10617</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/filewidgets/kdiroperator.cpp<br />
src/filewidgets/kfilewidget.cpp</div></div></div><br /><div><strong>To: </strong>ngraham, Frameworks, Dolphin, markg, elvisangelaccio<br /><strong>Cc: </strong>michaelh<br /></div>