D7446: Add a Recent Documents places item to Dolphin and file pickers by default

[Mark Gaiser]

markg requested changes to this revision.
markg added a comment.
This revision now requires changes to proceed.


  In https://phabricator.kde.org/D7446#154720, @ngraham wrote:
  
  > That said, it's not just me who wants this in Dolphin, too. There's a Bugzilla ticket with one dupe and 32 votes: https://bugs.kde.org/show_bug.cgi?id=357542
  >
  > Also, we have a semi-functional version of this already in the form of the Places panel's Recently Saved section. So there does seem to be some interest, both historical and current, in having Dolphin show recent files by default.
  
  
  Be aware that the thing you're trying to get approved here is **not** the fix for that bug!
  You'd have to dig in Baloo to fix that bug.
  
  You should not even be looking into the recentdocuments KIO slave as that is just not the part where the bug is (in this case).
  Also, the users' case is for the file open/save dialog, again not the main application.
  
  You yourself even said in the bug that this patch would only be a non optimal workaround. This is imho the most nasty thing; patches that "seem" to solve something but are in fact workarounds that merely hide the real problem. I admit, I - unknowingly - also did that in the past but was often corrected by someone. As you are now :)
  
  So, to make things clear.
  -1 for having this in dolphin' main window. Doesn't mean that it never gets in, just that i don't like it.
  +1 for having this on a contextual basis in the file/folder open/save dialogs, that would in fact be very beneficial! You should still look at Baloo for fixing it, not recentdocuments.

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D7446

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp, markg
Cc: andreaska, gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20171012/2bcf5e7b/attachment.htm>