D27851: Add rss icons in action

Noah Davis noreply at phabricator.kde.org
Wed Mar 4 23:53:30 GMT 2020


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200304/88b9ad03/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list