<table><tr><td style="">ndavis requested changes to this revision.<br />ndavis added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D29739">View Revision</a></tr></table><br /><div><div><p>I have to say, I'm very impressed overall. There are a few things I'd like to see changed though:</p>

<p>folder-documents should use a single page, not the copy icon <a href="https://phabricator.kde.org/F8320517" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8320517: Screenshot_20200514_153157.png</a> I suppose they should all have the corner at the top right, but that can be done another time.</p>

<p>folder-network is using a different style <a href="https://phabricator.kde.org/F8320527" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8320527: Screenshot_20200514_153705.png</a></p>

<p>folder-scripts is using a different style <a href="https://phabricator.kde.org/F8320536" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8320536: Screenshot_20200514_154402.png</a></p>

<p>panel elements in user-desktop should be more like the 32px version <a href="https://phabricator.kde.org/F8320540" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8320540: Screenshot_20200514_154711.png</a> The 64px version should be changed too, but that can be done another time.</p>

<h2 class="remarkup-header">Other observations (not required, just documenting them so they aren't forgotten)</h2>

<p>folder-games inner symbol looks a bit small, but that's going to be extra work, so I won't fault you if you'd rather do that later. The 16px version that you reused for the 48px symbol is a bit too small at 16px anyway, so that can be done another time.<br />
<a href="https://phabricator.kde.org/F8320524" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8320524: Screenshot_20200514_153422.png</a></p>

<p>Interestingly, document-print uses a different style for the smallest printer icon size. I think I prefer the document-print version, but that change can be done another time.<br />
<a href="https://phabricator.kde.org/F8320530" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8320530: Screenshot_20200514_153936.png</a> <a href="https://phabricator.kde.org/F8320532" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8320532: Screenshot_20200514_154148.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29739">https://phabricator.kde.org/D29739</a></div></div><br /><div><strong>To: </strong>manueljlin, VDG, ngraham, ndavis<br /><strong>Cc: </strong>ngraham, ndavis, kde-frameworks-devel, LeGast00n, cblack, michaelh, bruns<br /></div>