D14893: [recentdocuments:/] Filter out files that can't be browsed with a file manager
Nathaniel Graham
noreply at phabricator.kde.org
Thu Aug 16 23:30:22 BST 2018
ngraham created this revision.
ngraham added reviewers: broulik, Dolphin, Frameworks.
ngraham requested review of this revision.
REVISION SUMMARY
This is a more appropriate version of D13797: Only add actual documents to Recent Documents <https://phabricator.kde.org/D13797>. Basically we filter out anything that can't be browsed with a file manager, such as web linkns, `appstream://` urls, etc.
This ensures that the contents of the `recentdocuments:/` ioslave is relevant to the user.
TEST PLAN
Opened a link from Spectacle in my web browser. Without the patch, it shows up in `recentdocuments:/`. With the patch, it does not.
REPOSITORY
R320 KIO Extras
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D14893
AFFECTED FILES
recentdocuments/recentdocuments.cpp
To: ngraham, broulik, #dolphin, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180816/a7171de6/attachment.html>
More information about the Kde-frameworks-devel
mailing list