<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a reviewer: VDG.<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/D16694">View Revision</a></tr></table><br /><div><div><p>Thanks for the patch! And thanks for using <tt style="background: #ebebeb; font-size: 13px;">arc</tt>, too. That makes life a bit easier for us. :)</p>

<p>Couple of things:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">The title becomes the commit message, so please change it to something more descriptive, such as, "Improve visibility for Konsole icon when using Breeze Dark"</li>
<li class="remarkup-list-item">Please add <tt style="background: #ebebeb; font-size: 13px;">BUG: 367696</tt> to its own line in the summary section and add some explanation there regarding why this patch is necessary.</li>
<li class="remarkup-list-item">The <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Test Plan</span></span></span> section is empty; we need some evidence of testing. Can you provide some screenshots that show the icon when using Breeze Dark?</li>
<li class="remarkup-list-item">In the future, please tag <a href="https://phabricator.kde.org/tag/vdg/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">#VDG</a> for anything that involve an icon or visual change.</li>
</ol>

<p>See also <a href="https://community.kde.org/Infrastructure/Phabricator#Formatting_your_patch" class="remarkup-link" target="_blank" rel="noreferrer">https://community.kde.org/Infrastructure/Phabricator#Formatting_your_patch</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/D16694">https://phabricator.kde.org/D16694</a></div></div><br /><div><strong>To: </strong>anishgiri, ngraham, VDG<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>