<table><tr><td style="">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/D18673">View Revision</a></tr></table><br /><div><div><p>I like the functionality, though I'm not thrilled about having to change these symlinks around as apps change their appstream IDs, and constabtly add new ones as different packagers replace the existing AppStream IDs with their own (ugh).  Then again that's already the world we live in and I guess we've signed ourselves up for this mess by supporting icons for 3rd-party apps anyway.</p>

<p>This currently causes the <tt style="background: #ebebeb; font-size: 13px;">scalable</tt> test to fail, so at a minimum that needs to be fixed before this patch can land:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">FAIL!  : ScalableTest::test_scalable(icons:Applications) The following icons are not available in a scalable directory:
  com.github.bajoja.indicator-kdeconnect
   Loc: [/home/dev/kde/src/breeze-icons/autotests/scalabletest.cpp(262)]
FAIL!  : ScalableTest::test_scalable(icons-dark:Applications) The following icons are not available in a scalable directory:
  com.github.bajoja.indicator-kdeconnect</pre></div>

<p>Remove the 16px <tt style="background: #ebebeb; font-size: 13px;">com.github.bajoja.indicator-kdeconnect</tt> icon symlink, I guess.</p>

<p>Looks like <a href="https://phabricator.kde.org/p/nibags/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@nibags</a> found another one that could be added, too</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/D18673">https://phabricator.kde.org/D18673</a></div></div><br /><div><strong>To: </strong>GeeXT, Frameworks, VDG, ngraham<br /><strong>Cc: </strong>ngraham, nibags, kde-frameworks-devel, michaelh, bruns<br /></div>