D11768: Add Desktop and Downloads to the default list of Places
Mark Gaiser
noreply at phabricator.kde.org
Thu Mar 29 16:32:22 UTC 2018
markg requested changes to this revision.
markg added a comment.
This revision now requires changes to proceed.
I don't know why, but i already have this by default in my Dolphin... (ArchLinux user here, i don't think they add it explicitly as they try to stay as true to upstream as possible).
So, err, don't know why.
Anyhow, having them by default is a big +1 from me.
Yet i still give a -1.. The reason for that is simple. Ever since the placesmodel changes in KIO and Dolphin, a unittest for that fails. Fix that first.
I did have a look at it, but quite some placesmodel magic changed so it's better if one of the authors of those changes takes a look.
For reference:, look at all the recent builds of Dolphin and KIO.
Dolphin: https://build.kde.org/job/Applications%20dolphin%20kf5-qt5%20SUSEQt5.9/
KIO: https://build.kde.org/job/Frameworks%20kio%20kf5-qt5%20SUSEQt5.10/
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D11768
To: ngraham, #dolphin, #gwenview, #frameworks, #vdg, markg
Cc: markg, fabiank, progwolff, broulik, michaelh, ngraham
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180329/6bb24467/attachment.html>
More information about the Kde-frameworks-devel
mailing list