<table><tr><td style="">ngraham added reviewers: cfeck, broulik, elvisangelaccio.<br />ngraham requested changes to this revision.<br />ngraham 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/D17623">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D17623#378023" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D17623#378023</a>, <a href="https://phabricator.kde.org/p/ndavis/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ndavis</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I'd have to make PRs for ~60 GitHub repositories, so I'd like to avoid that if possible.</p></div>
</blockquote>

<p>I understand that this would be really annoying, but conceptually it seems like the correct approach. If YaST wants to icons from icon themes, it doesn't seem reasonable to expect every single icon theme in existence to make special icons just fro YaST. YaST should use commonly-named icons.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>When the time comes, we can make symlinks with more appropriate names and different sizes.</p></blockquote>

<p>If we continue to use this patch to create icons with names that start with <tt style="background: #ebebeb; font-size: 13px;">yast-</tt>, I would prefer for those to be the symlinks. So for example your new <tt style="background: #ebebeb; font-size: 13px;">yast-auth-client</tt> icon should be named <tt style="background: #ebebeb; font-size: 13px;">auth-kerberos</tt> and <tt style="background: #ebebeb; font-size: 13px;">yast-auth-client</tt> should be a symlink to it.</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/D17623">https://phabricator.kde.org/D17623</a></div></div><br /><div><strong>To: </strong>ndavis, VDG, cfeck, broulik, elvisangelaccio, ngraham<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>