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