D22653: Add view-barcode-qr icons

Noah Davis noreply at phabricator.kde.org
Tue Jul 30 12:35:25 BST 2019


ndavis added a comment.


  In D22653#504138 <https://phabricator.kde.org/D22653#504138>, @lavender wrote:
  
  > I noticed that only the 22px icons use the viewbox, is this intentional?
  >
  > As for the 32px one the validator <https://validator.nu/> I used complains that:
  >
  >   Error: Attribute paint-order not allowed on SVG element path at this point.
  >  
  >       From line 7, column 5; to line 7, column 1521
  >  
  >       tyle>↩    <path class="ColorScheme-Text" fill="currentColor" style="fill:currentColor;fill-opacity:1;stroke:no… 16v-2h16v2zm16 0h-2V3h2zm0-16v2h-16V3z" paint-order="markers fill stroke" transform="rotate(-90)"/>↩</s
  >
  
  
  viewbox vs height/width doesn't matter, the result is the same. the validator might be complaining about the stylesheets, but idk. what really matters is whether or not it looks right in the Qt SVG renderer.

REPOSITORY
  R266 Breeze Icons

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

To: mbruchert, #breeze, ndavis
Cc: lavender, ngraham, broulik, ndavis, kde-frameworks-devel, LeGast00n, sbergeron, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190730/1e2457a9/attachment.html>


More information about the Kde-frameworks-devel mailing list