trickyricky26 added a comment.

  Visually I think this is fine and a good improvement.
  
  I wonder why your SVGs have two seperate paths as opposed to unifying them 
into one path (as I usually do)? Just curious if there is any benefit to using 
one approach above the other.
  Also, judging by the MDN Page 
<https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/fill-rule> I don't 
think the `fill-rule="evenodd"` does anything here.

REPOSITORY
  R266 Breeze Icons

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

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

Reply via email to