ndavis added a comment.
I see you've kept the green on the light theme version instead of using the same icon for both versions. Is there a reason for this or do you just prefer it? This looks just about ready to land on the master branch, there are just a few more things you need to do. INLINE COMMENTS > code.svg:903 > + height="10.583333" /> > + <rect > + > style="opacity:0.35;fill:none;fill-opacity:1;stroke:none;stroke-width:0.26458332" Delete this invisible rectangle. > code.svg:884 > + height="10.583333" /> > + <rect > + > style="opacity:0.35;fill:none;fill-opacity:1;stroke:none;stroke-width:0.26458332" Delete this invisible rectangle. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D19324 To: axionl, #vdg Cc: ndavis, rooty, ngraham, kde-frameworks-devel, michaelh, bruns