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: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180816/a7171de6/attachment.htm>


More information about the kfm-devel mailing list