<table><tr><td style="">ndavis added a comment.
</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/D25015">View Revision</a></tr></table><br /><div><div><p>I noticed a few things.</p>

<hr class="remarkup-hr" />

<p>The masks have a rather odd shape and it's not pixel aligned, which might mean something is wrong: <br />
<a href="https://phabricator.kde.org/F7891523" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7891523: Screenshot_20200118_225818.PNG</a><br />
It's different from master, which looks pretty normal: <br />
<a href="https://phabricator.kde.org/F7891527" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7891527: Screenshot_20200118_230209.PNG</a></p>

<hr class="remarkup-hr" />

<p>In the <tt style="background: #ebebeb; font-size: 13px;">panel-background</tt> a <tt style="background: #ebebeb; font-size: 13px;">tooltip</tt> SVGs the shadow was changed, but the tooltip was not. Keep in mind that <br />
<a href="https://phabricator.kde.org/F7891542" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7891542: Screenshot_20200118_231610.PNG</a></p>

<hr class="remarkup-hr" />

<p>I get the impression that these were changed with the node tool rather than creating the corners from rounded rectangles. I guess that works as long as all the svg corners are consistent with each other, but it's inconsistent with the QStyle and could still lead to visible pixel misalignment when the UI is scaled up.<br />
<a href="https://phabricator.kde.org/F7891536" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7891536: Screenshot_20200118_230921.PNG</a></p>

<hr class="remarkup-hr" />

<p>Why is <tt style="background: #ebebeb; font-size: 13px;">widgets/background.svg</tt> different from the other ones? Maybe there's a good reason, since it's also different in the master branch, but its very odd.<br />
<a href="https://phabricator.kde.org/F7891628" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7891628: Screenshot_20200118_232026.PNG</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>BRANCH</strong><div><div>bettershadows (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25015">https://phabricator.kde.org/D25015</a></div></div><br /><div><strong>To: </strong>niccolove, VDG, ngraham<br /><strong>Cc: </strong>mart, broulik, cblack, ndavis, manueljlin, ngraham, filipf, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns<br /></div>