D23742: Add two special cases url recentlyused:/files and recentlyused:/folders

Méven Car noreply at phabricator.kde.org
Tue Sep 10 20:28:53 BST 2019


meven added inline comments.

INLINE COMMENTS

> recentlyused.cpp:98
> -    // handles type aka mimetype
> -    if (urlQuery.hasQueryItem(QStringLiteral("type"))) {
> -        const auto typeValue = urlQuery.queryItemValue(QStringLiteral("type"));

Should we allow the filtering through type with /files path ?
I am thinking I should change that.

REPOSITORY
  R320 KIO Extras

BRANCH
  arcpatch-D23742

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

To: meven, ivan, ngraham, #frameworks
Cc: kde-frameworks-devel, kfm-devel, iasensio, fprice, LeGast00n, MrPepe, fbampaloukas, alexde, GB_2, Codezela, feverfew, meven, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190910/9357ec0f/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list