<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: Dolphin, Frameworks, broulik.<br />Restricted Application added a project: Frameworks.<br />Restricted Application added a subscriber: kde-frameworks-devel.<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/D13797">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>KIO's Recent Documents feature is not as useful as it could be right now because KRun and the KURl dialog unconditionally add all URLs to it, so it tends to fill up with web links, appstream:/ URLs, and the like.</p>

<p>This patch fixes that by having them only append a URL to Recent Documents if it's actually a document.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Compiled, deployed, and rebooted.</p>

<p>Used KRunner to go to <a href="https://kde.org" class="remarkup-link" target="_blank" rel="noreferrer">https://kde.org</a> and several other websites and then navigated to `recentdocuments:/ in Dolphin. No web URLs appeared in the list of recent documents.</p>

<p>Opened a random file in Dolphin and then went to <tt style="background: #ebebeb; font-size: 13px;">recentdocuments:/</tt>; the file still showed up there, as expected.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>recent-documents-only-shows-actual-documents (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13797">https://phabricator.kde.org/D13797</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/widgets/krun.cpp<br />
src/widgets/kurlrequesterdialog.cpp</div></div></div><br /><div><strong>To: </strong>ngraham, Dolphin, Frameworks, broulik<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>