Only add actual documents to Recent Documents
AbandonedPublic

Authored by ngraham on Jun 29 2018, 5:27 PM.

Details

Reviewers
broulik
Group Reviewers
Dolphin
Frameworks
Summary

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.

This patch fixes that by having them only append a URL to Recent Documents if it's actually a document.

Test Plan

Compiled, deployed, and rebooted.

Used KRunner to go to https://kde.org and several other websites and then navigated to `recentdocuments:/ in Dolphin. No web URLs appeared in the list of recent documents.

Opened a random file in Dolphin and then went to recentdocuments:/; the file still showed up there, as expected.

Diff Detail

Repository
R241 KIO
Branch
recent-documents-only-shows-actual-documents (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 457
Build 457: arc lint + arc unit