ndavis accepted this revision.
ndavis added a comment.
This revision is now accepted and ready to land.


  In D27851#622296 <https://phabricator.kde.org/D27851#622296>, @cblack wrote:
  
  > SVG-wise, optimization here seems suboptimal. There's excess elements that 
could be removed.
  >
  > Visually, the icon looks like the RSS icon, which is about all you need out 
of an RSS icon.
  >
  > I do believe we need at least 16px and 22px for monochromatic actions, 
though.
  
  
  The main reason we optimize is to make the code readable, not to make it as 
small as possible. The current version is fine.

REPOSITORY
  R266 Breeze Icons

BRANCH
  master

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

To: ognarb, #vdg, #breeze, ndavis
Cc: ndavis, cblack, kde-frameworks-devel, LeGast00n, fbampaloukas, GB_2, 
trickyricky26, michaelh, ngraham, bruns

Reply via email to