<table><tr><td style="">ndavis added a subscriber: niccolove.<br />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/D26955">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/D26955#601684" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D26955#601684</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>This fixes the color bug, so that's good. :)</p>

<p>However with Breeze Dark, now I see pointy corners and very weak shadows: <a href="https://phabricator.kde.org/F7982216" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7982216: Screenshot_20200127_221203.png</a></p>

<p>...Whereas with the Breeze and Breeze Light plasma style, I see nice shadows and rounded corners : <a href="https://phabricator.kde.org/F7982219" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7982219: Screenshot_20200127_221718.png</a></p>

<p>(and yes I cleared the plasma SVG cache)</p></div>
</blockquote>

<p>Did you test the original shadows patch with breeze dark? These shadows are identical. I even zoomed into screenshots 1600% and used a color picker on different pixels in the shadow. If you don't like these shadows then we should just revert the shadows patch instead of fixing them with this patch.</p>

<p>I don't know what to do about the corners. There's no reason why they should have sharp corners. Even if I replicate the previous corners, the issue is still there. I'm suspecting a bug in the code. <a href="https://phabricator.kde.org/p/niccolove/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@niccolove</a> also had a lot of trouble with the mask. He said it doesn't seem to even work and deleting the mask from the SVG has no visible effect. I'm not even sure if the mask is the cause of the corners. Even the shadows patch still has some visual artifacts in the corners: <a href="https://phabricator.kde.org/F7982336" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7982336: Screenshot_20200128_004638.PNG</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26955">https://phabricator.kde.org/D26955</a></div></div><br /><div><strong>To: </strong>ndavis, VDG, Plasma<br /><strong>Cc: </strong>niccolove, ngraham, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns<br /></div>