D16772: Add new icons for root
Nathaniel Graham
noreply at phabricator.kde.org
Fri Nov 9 21:50:35 GMT 2018
ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.
Makes sense to me. Now that I see them all together, I agree that /_ is a bit better. It seems much better for the folder, and at least no worse (and probably a bit better) for the disk.
Any other VDG comments, or should we land this?
BTW, @ndavis, you're doing so much good work that I would recommend you apply for a Developer account so you can land your own patches. You can read about that here: https://community.kde.org/Infrastructure/Get_a_Developer_Account
BTW, once this goes in, I'll submit a KIO patch to use the new icon for the current Root Places Panel item. If we end up not removing it after all, we might as well use the nice new icon! :)
REPOSITORY
R266 Breeze Icons
BRANCH
folder-root (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D16772
To: ndavis, #vdg, ngraham
Cc: ngraham, #vdg, kde-frameworks-devel, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181109/93ea266e/attachment.html>
More information about the Kde-frameworks-devel
mailing list