<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/D10212" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Hmm, something's not right here. I applied the patch, and the dark diagonal shadows are missing. Also, <tt style="background: #ebebeb; font-size: 13px;">arc</tt> complained that the base commit was not in the repo, so I suspect the diff was still not generated correctly</p>

<p>At this point probably the sanest thing to do is as follows:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Save a copy of the icon elsewhere</li>
<li class="remarkup-list-item">Revert the breeze-icons git repo to a pristine state: <tt style="background: #ebebeb; font-size: 13px;">git reset --hard origin/master</tt></li>
<li class="remarkup-list-item">Replace <tt style="background: #ebebeb; font-size: 13px;">virtualbox.svg</tt> in the now-clean repo with your new copy</li>
<li class="remarkup-list-item">Generate a new diff <tt style="background: #ebebeb; font-size: 13px;">git diff</tt> and use that here.</li>
</ol>

<p>Then after this, we'll get you set up with <tt style="background: #ebebeb; font-size: 13px;">arc</tt> so it's easy. :)</p>

<p>Also, in actual use, I have to say that I'm not a fan of the huge white border. I think it would look better without that.</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10212" rel="noreferrer">https://phabricator.kde.org/D10212</a></div></div><br /><div><strong>To: </strong>lshoravi, Breeze, ngraham, VDG<br /><strong>Cc: </strong>davidc, Frameworks, michaelh, ngraham<br /></div>