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

Reply via email to