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

[Nathaniel Graham]

ngraham created this revision.
ngraham added a project: Dolphin.
Restricted Application added a subscriber: Konqueror.

REVISION SUMMARY
BUG: 357542

Add a Recent Documents places item (recentdocuments:/) to Dolphin by default.

TEST PLAN
Tested this diff in up-to-date KDE Neon. By default, a recentdocuments:/ places item appears:

F3870935: Recent Documents.png <https://phabricator.kde.org/F3870935>

REPOSITORY
R318 Dolphin

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

AFFECTED FILES
src/panels/places/placesitemmodel.cpp

To: ngraham, #dolphin, #kde_applications
Cc: elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, firef, andrebarros, emmanuelp

[Elvis Angelaccio]

elvisangelaccio added a comment.


I don't know, this is not really a "place". I would expect to find such an item in the "Recently Saved" section (even though `recentdocuments:/` seems to only show recently *opened* files).
What I'd like is an actually functional `timeline:/` ioslave... :/

REPOSITORY
R318 Dolphin

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

To: ngraham, #dolphin, #kde_applications
Cc: elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added a comment.


I made it a bookmark for the following reasons:

1. It was a small, focused, low-risk change
2. That also has the effect of automatically adding it to the Places panel in file pickers, which is part of the feature request. If we put this in the Search For section, that would involve much more work and risk, both to add it there and also to modify the file pickers to show that section.

REPOSITORY
R318 Dolphin

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

To: ngraham, #dolphin, #kde_applications
Cc: elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, firef, andrebarros, emmanuelp

[Kai Uwe Broulik]

broulik added a comment.


I would prefer this item to either go *before* Trash or after Home. Also, can this be done in `KFilePlacesModel`, so it also shows up in the file dialog?

REPOSITORY
R318 Dolphin

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

To: ngraham, #dolphin, #kde_applications
Cc: broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added a comment.


I haven't forgotten about this, but life has intervened and I'm not going to have time to work on it for a few days, maybe even a couple of weeks. If anyone else wants to run with it and take over, I won't be in the least bit offended. If not, I'll resume work at some point in the near to medium term future.

REPOSITORY
R318 Dolphin

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

To: ngraham, #dolphin, #kde_applications
Cc: broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham retitled this revision from "Add a Recent Documents places item to Dolphin by default" to "Add a Recent Documents places item to Dolphin and file pickers by default".
ngraham edited the summary of this revision.
ngraham changed the repository for this revision from R318 Dolphin to R241 KIO.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications
Cc: broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham updated this revision to Diff 18867.
ngraham added a comment.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.


This update implements the following requested changes:

1. Located after Home bookmark and before Trash bookmark
2. Moved to KIO's `kfileplacesmodel.cpp`, which makes it appear in file pickers as well

REPOSITORY
R241 KIO

CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7446?vs=18468&id=18867

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

AFFECTED FILES
src/filewidgets/kfileplacesmodel.cpp

To: ngraham, #dolphin, #kde_applications
Cc: #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham edited the test plan for this revision.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications
Cc: #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added reviewers: broulik, elvisangelaccio.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio
Cc: #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, nicolasfella, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added a comment.


Any remaining objections or requested changes?

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio
Cc: #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, nicolasfella, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added reviewers: dfaure, emmanuelp.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added a comment.


Anybody object?

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, akrutzler, navarromorales, firef, andrebarros, emmanuelp

[Mark Gaiser]

markg added a comment.


You could continue and add about a dozen more "semi-useful" protocol links (ftp, settings, programs, bluetooth, and much more) ;) .
I don't think that's the route for the places panel to go. Not by default that is.
If the user wants it, it can be added just like any shortcut can be added in the places thingy. By default it should stay rather clean. recentdocuments is (in my opinion) not one that should be there by default. Also, for me personally it seems rather weird as recently visited url's are also in the recent documents.... And files I've accessed on the console don't appear in it at all (understandable though).

However, it would be a nice candidate for a custom panel! I would have guessed there'd be a panel for that, but there isn't. So a new panel with recent documents in it (that the user can just turn on or off, off by default!) sounds much more valuable then a mere link.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, akrutzler, navarromorales, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added a comment.


I don't think this is "semi-useful". " A Recent Documents feature in the file manager and open/save dialogs is IMHO really important, especially for lesst-technical users who use features like this on other platforms expensively instead of making extensive use of folder hierarchies. "The user can add it" is a problematic response since 99.9% of users don't know this exists, and therefore don't know that this functionality is available in the first place (and if they did, they would find it challenging to add).

The file managers and open/save dialogs in macOS and GNOME both have some kind of "recent stuff" feature shown by default, and it's really nice. And ad evidenced by the but report and the fact that people have voted for it, this is a desirable feature to some of our users.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, akrutzler, navarromorales, firef, andrebarros, emmanuelp

[Mark Gaiser]

markg added a comment.

Post by Nathaniel Graham
I don't think this is "semi-useful". " A Recent Documents feature in the file manager and open/save dialogs is IMHO really important, especially for lesst-technical users who use features like this on other platforms expensively instead of making extensive use of folder hierarchies. "The user can add it" is a problematic response since 99.9% of users don't know this exists, and therefore don't know that this functionality is available in the first place (and if they did, they would find it challenging to add).
The file managers and open/save dialogs in macOS and GNOME both have some kind of "recent stuff" feature shown by default, and it's really nice. And ad evidenced by the but report and the fact that people have voted for it, this is a desirable feature to some of our users.

As i said, as a "link" i see no use for it.
As a separate panel, i do see a use for it, but still not enabled by default imho.

Also, really, look at the output of recentdocuments:/ In my case i have the following issues:

1. It shows a folder. (my home downloads folder). Clicking it shows a "The file or folder recentdocuments:/Downloads does not exist." (it's text is /hom/mark/Downloads which does exist on my local drive! Also, why does a folder show up in recentdocuments amyhow, it's not a "document".
2. just 1 file in the list! (i only have 9 entries, guess i don't open much files, hehe)
3. Everything else is web links
4. The error as said in 1. can be closed however many times you want, it keeps reappearing if you press https://phabricator.kde.org/F5 or enter "recentdocuments:/" again. It only goes away if you close the error, go to another location and then to recentdocuments:/

How can you possibly want to present that to the user by default? Let me put it this way, the ioslave "probably" hasn't seen much development. It needs some love to be suitable for a real recent documents and even be considered to put in the places panel.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, akrutzler, navarromorales, firef, andrebarros, emmanuelp

[Elvis Angelaccio]

elvisangelaccio added a comment.


Yeah I also find weird to see weblinks in there. Maybe if we call it "History" would be a bit better (and it would also match the History tab in Kickoff).

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, akrutzler, navarromorales, firef, andrebarros, emmanuelp

[Mark Gaiser]

markg added a comment.

Post by Elvis Angelaccio
Yeah I also find weird to see weblinks in there. Maybe if we call it "History" would be a bit better (and it would also match the History tab in Kickoff).

Perhaps, but then we'd still miss a real "recentdocuments" :)

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, akrutzler, navarromorales, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added a comment.


It seems odd to have all of these special KIO URLs that we don't actually want to use because they're rough and underdeveloped. They're rough and underdeveloped because they're hidden by default, so nobody sees them, and nobody files bugs or submits patches for them. But I do see your point.

That said, the advantage to adding this as a Places item is that it shows up in file open/save dialogs for free, which is where it's most useful. If we make this into a whole new panel, we'll have to do a bunch of otherwise unnecessary special work to get it into open/save dialogs. I don't see the advantage.

If the objection is that the content isn't useful (why does it show URLs?), then I can fix that too, but only if by doing so, folks will be amenable to adding a "Recent Documents" entry by default. It really is useful to have recent documents aggregated somewhere.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, akrutzler, navarromorales, firef, andrebarros, emmanuelp

[gregormi]

gregormi added a comment.

Post by Nathaniel Graham
That said, the advantage to adding this as a Places item is that it shows up in file open/save dialogs for free, which is where it's most useful.

This would support the use case presented here: https://store.kde.org/p/1156273 and here: https://bugs.kde.org/show_bug.cgi?id=384411

Regarding those use cases, are you planning to add a list of recent folders to the Select Folder dialog, too?

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, akrutzler, navarromorales, firef, andrebarros, emmanuelp

[Mark Gaiser]

markg added a comment.

Post by Nathaniel Graham
It seems odd to have all of these special KIO URLs that we don't actually want to use because they're rough and underdeveloped. They're rough and underdeveloped because they're hidden by default, so nobody sees them, and nobody files bugs or submits patches for them. But I do see your point.
That said, the advantage to adding this as a Places item is that it shows up in file open/save dialogs for free, which is where it's most useful. If we make this into a whole new panel, we'll have to do a bunch of otherwise unnecessary special work to get it into open/save dialogs. I don't see the advantage.
If the objection is that the content isn't useful (why does it show URLs?), then I can fix that too, but only if by doing so, folks will be amenable to adding a "Recent Documents" entry by default. It really is useful to have recent documents aggregated somewhere.

Regarding the advantage in the file open/save dialog. It's only a advantage because the places panel is there by default.
A recently used files/documents entry just doesn't belong in there (in my opinion).
So you would have to add a second panel for that. It is the best looking way, but yeah, requires quite a bit more code changes.

Recent documents just doesn't fit the open/save dialog imho. How useful is it to see the recently accessed files there? Not very. What you would want (and i would in fact like to have that) is a recent folders in open/save. That would be very useful! A IO slave for that does not exist.
In dolphin a "recentfiles" panel would be interesting (i still wouldn't like it on by default..), but as it currently stands, no IO slave can currently do that. You would either have to "fix" recentdocuments to be that or make a new one.

Also note that everything i said is merely my opinion. I'm not blocking you (i don't feel like i have the right to with my lack of contributions in recent years). If more people find it useful the way you intend it and if there is no comment on the code change then you are quite simply allowed to commit. But in this case you'd probably want to wait (or ask) for a few more opinions before making that decision. All i'm trying to do is provide constructive feedback :)

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, akrutzler, navarromorales, firef, andrebarros, emmanuelp

[Andreas Kainz]

andreaska added a comment.


works for the VDG really good job. well done. I like it.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: andreaska, gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, spoorun, navarromorales, firef, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added a comment.


@markg, I strongly disagree that "recent files" are not useful in open/save dialogs.

As I've mentioned before, a common use case that this enables is when you save an item from one program and need to open it in another without having a Dolphin window open that shows it. One example is attaching files to websites. You just go to the "Recent files" entry in the Open window and voila, your latest saved file is right there on the top. It's a super fast workflow.

There is a reason why "Recent items" is a default entry in the file manager and open/save dialogs in macOS and GNOME. They didn't add it for no good reason.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: andreaska, gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, spoorun, navarromorales, firef, andrebarros, emmanuelp

[Mark Gaiser]

markg added a comment.

Post by Nathaniel Graham
@markg, I strongly disagree that "recent files" are not useful in open/save dialogs.
As I've mentioned before, a common use case that this enables is when you save an item from one program and need to open it in another without having a Dolphin window open that shows it. One example is attaching files to websites. You just go to the "Recent files" entry in the Open window and voila, your latest saved file is right there on the top. It's a super fast workflow.
There is a reason why "Recent items" is a default entry in the file manager and open/save dialogs in macOS and GNOME. They didn't add it for no good reason.

As you seem eager to compare against others who do have it. How about others who don't. That's for instance XFCE with Thunar, it doesn't have that.
Windows 7 had is partly, called "Recent places" which only lists folders, not files.
Windows 10 doesn't have it.
macOS finder - contrary to what you keep telling here - does not show recent documents or folders by default.

All of the above is for the main applications. I'm not entirely sure about the file/folder open dialogs.

The only way i could find this convenient is if it were by context and the file open dialog boxes only. What i mean by that is when you get a file open dialog box and need to select a file then having the recently used files in there might be neat.
But by default in dolphin' main view in the favorites section... No, that doesn't seem the appropriate place.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: andreaska, gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added a comment.


I could get behind only adding it to Open File dialogs. Let me have a think on that.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: andreaska, gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added a comment.


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.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp
Cc: andreaska, gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp

[Mark Gaiser]

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

Post by Nathaniel Graham
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

[Anthony Fieroni]

anthonyfieroni added a comment.


You can add recent documents as an action like in Kate. Places model looks in not correct destination.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp, markg
Cc: anthonyfieroni, andreaska, gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham added a comment.

Post by Anthony Fieroni
You can add recent documents as an action like in Kate. Places model looks in not correct destination.

I'm not sure exactly what this is referring to. Can you clarify?

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp, markg
Cc: anthonyfieroni, andreaska, gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp

[Störm Poorun]

spoorun added a comment.


I support having the recently used option, perhaps by default in the open/save dialogue, and also as an optional panel.
This would be highly useful for workflow for many, including using other software as suggested, but also coming back to documents (in multiple locations) being referred to (though not necessarily saved, so not appearing in the 'recently saved' section.

(It's unfortunate that there's not a user-friendly way of selecting components of the Places panel).

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp, markg
Cc: spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, firef, ngraham, andrebarros, emmanuelp

[Anthony Fieroni]

anthonyfieroni added a comment.

Post by Nathaniel Graham
I'm not sure exactly what this is referring to. Can you clarify?

https://api.kde.org/frameworks/kconfigwidgets/html/classKRecentFilesAction.html

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp, markg
Cc: spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, #frameworks, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, navarromorales, firef, ngraham, andrebarros, emmanuelp

[Nathaniel Graham]

ngraham updated this revision to Diff 42277.
ngraham added a comment.
Herald edited subscribers, added: kde-frameworks-devel; removed: Frameworks.


Re-base on master

REPOSITORY
R241 KIO

CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7446?vs=18867&id=42277

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

AFFECTED FILES
src/filewidgets/kfileplacesmodel.cpp

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp, markg
Cc: kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, #frameworks

[Nathaniel Graham]

ngraham retitled this revision from "Add a Recent Documents places item to Dolphin and file pickers by default" to "[Places panel] Add a Recently Used item by default".
ngraham edited the summary of this revision.
ngraham edited the test plan for this revision.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp, markg
Cc: kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Nathaniel Graham]

ngraham added a task: T8349: Improve Places panel usability and presentation.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp, markg
Cc: kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Nathaniel Graham]

ngraham edited the test plan for this revision.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp, markg
Cc: kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Nathaniel Graham]

ngraham added a comment.


I'd like to re-submit this patch (now re-based on master) for consideration. I put some work into improving the `recentdocuments:/` ioslave in D14893 <https://phabricator.kde.org/D14893> so it's useful now. User requests for this feature to be added by default still pop up from time to time, and both GNOME and macOS both have this feature in their file browsers and file pickers. I think it's relevant and desirable.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, #kde_applications, broulik, elvisangelaccio, dfaure, emmanuelp, markg
Cc: kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Nathaniel Graham]

ngraham edited reviewers, added: VDG, Frameworks; removed: KDE Applications, dfaure, emmanuelp.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks, #kde_applications
Cc: kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Sven Mauch]

svenmauch added a comment.


While I wouldn't use it (yet?) I think it's a great addition and would benefit a lot of people. I give it a +1, especially if D15739 <https://phabricator.kde.org/D15739> gets approved aswell. ;)

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Nathaniel Graham]

ngraham added a comment.


Ping!

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Nathaniel Graham]

ngraham edited the summary of this revision.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[David Edmundson]

davidedmundson added a comment.


This is in frameworks, recentdocuments is in kio-extras.

Are you sure you can do this?

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Nathaniel Graham]

ngraham added a comment.

Post by David Edmundson
This is in frameworks, recentdocuments is in kio-extras.
Are you sure you can do this?

Not sure I catch your meaning. The `recentdocuments:/` kioslave has been shipped since forever; this patch just adds it to the Places panel by default.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[David Faure]

dfaure added a comment.


I guess the implicit part of the question is "what if kio-extras isn't installed ?".
I assume it will lead to a broken item.

So this should probably use a check like `KProtocolInfo::isKnownProtocol("recentdocuments")`

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Nathaniel Graham]

ngraham added a comment.

Post by David Faure
I guess the implicit part of the question is "what if kio-extras isn't installed ?".
I assume it will lead to a broken item.
So this should probably use a check like `KProtocolInfo::isKnownProtocol("recentdocuments")`

Ah, I see now. Sure, I can do that.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Nathaniel Graham]

ngraham updated this revision to Diff 42865.
ngraham added a comment.


Only create "Recently Used" if the protocol is available because `kio-extras` is installed

REPOSITORY
R241 KIO

CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7446?vs=42277&id=42865

BRANCH
arcpatch-D7446

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

AFFECTED FILES
src/filewidgets/kfileplacesmodel.cpp

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Nathaniel Graham]

ngraham edited the summary of this revision.
ngraham added a dependency: D15739: [Places panel] Don't show Root by default.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, alexde, sourabhboss, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp

[Nathaniel Graham]

ngraham updated this revision to Diff 45040.
ngraham added a comment.


- Revamp the whole section

REPOSITORY
R241 KIO

CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7446?vs=42865&id=45040

BRANCH
arcpatch-D7446

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

AFFECTED FILES
src/filewidgets/kfileplacesitem.cpp
src/filewidgets/kfileplacesmodel.cpp

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: huftis, svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, alexde, sourabhboss, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov

[Andreas Kainz]

andreask added a comment.


I would suggest to have by default detail view instead of icon view cause with detail view it's easier to read the paths.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: andreask, huftis, svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, alexde, sourabhboss, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov

[Nathaniel Graham]

ngraham updated this revision to Diff 45043.
ngraham added a comment.


Rebase on master

Updating D7446 <https://phabricator.kde.org/D7446>: [Places panel] Add a Recently Used item by default

REPOSITORY
R241 KIO

CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7446?vs=45040&id=45043

BRANCH
arcpatch-D7446

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

AFFECTED FILES
src/filewidgets/kfileplacesitem.cpp
src/filewidgets/kfileplacesmodel.cpp

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: andreask, huftis, svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, alexde, sourabhboss, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov

[Nathaniel Graham]

ngraham retitled this revision from "[Places panel] Add a Recently Used item by default" to "[Places panel] Revamp the Recently Saved section".
ngraham edited the summary of this revision.
ngraham edited the test plan for this revision.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: andreask, huftis, svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, alexde, sourabhboss, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov

[Nathaniel Graham]

ngraham added a comment.

Post by Andreas Kainz
I would suggest to have by default detail view instead of icon view cause with detail view it's easier to read the paths.

That's a good idea, let me see what I can do.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: andreask, huftis, svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, alexde, sourabhboss, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov

[Nathaniel Graham]

ngraham added a comment.


Ping! #VDG <https://phabricator.kde.org/tag/vdg/>, #dolphin <https://phabricator.kde.org/tag/dolphin/>, or #frameworks <https://phabricator.kde.org/tag/frameworks/> folks?

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: andreask, huftis, svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, alexde, sourabhboss, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov

[TrickyRicky]

trickyricky26 added a comment.


I like this change a lot, however I think the Section header should be changed from "Recents" to "Recently Used". I think Recents just sound inelegant and weird

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: trickyricky26, andreask, huftis, svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, alexde, sourabhboss, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov

[Nathaniel Graham]

ngraham added a comment.


Using "Recently Used" as the section header label presents the following problems:

- One of the items is named "Recently Used"
- The other items actually do not display a list of recently used files; they display a list of recently //saved// files.

That's why I was trying to come up with a generic section header label that could feasible encompass both.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: trickyricky26, andreask, huftis, svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, alexde, sourabhboss, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov

[David Edmundson]

davidedmundson added a comment.


Note also that Kickoff's "History" with recent files is /yet another/ completely different data source.

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: trickyricky26, andreask, huftis, svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, alexde, sourabhboss, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov

[Elvis Angelaccio]

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


We need to update `kfileplacesmodeltest` (as well as `placesitemmodeltest` in dolphin).

REPOSITORY
R241 KIO

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

To: ngraham, #dolphin, broulik, elvisangelaccio, markg, #vdg, #frameworks
Cc: trickyricky26, andreask, huftis, svenmauch, kde-frameworks-devel, spoorun, anthonyfieroni, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, #konqueror, alexde, sourabhboss, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov