<table><tr><td style="">rooty 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/D18545">View Revision</a></tr></table><br /><div><div><p>Nate, thank you for accepting the patch!</p>

<p>Okay so the corner thing - that bug actually happens <em>everywhere.</em><br />
<strong>All of the following screenshots are of master, not this patch.</strong></p>

<p>For example, Breeze Dark, Background Contrast on, Blur off <a href="https://phabricator.kde.org/F6580832" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6580832: fook-definitive-noeffects.png</a></p>

<p>I apologize for the small screenshot but if you zoom in you'll see that there's a square at the center that's #2C2E31 and the rest of the notification is #2E3235. The colors should be the same but they're not.</p>

<p>If you turn the blur on, it still exhibits the same issue.<br />
If you turn background contrast off, the square becomes brighter, but still different from the rest of the notification.<br />
<a href="https://phabricator.kde.org/F6580839" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6580839: image.png</a></p>

<p>Seeing as all of these screenshots are the current master, what this patch seems to do for this issue is make it more visible, but it doesn't seem to be causing it.  Manipulating the black rectangle (alpha mask) doesn't seem to fix any of these issues so I don't think that <em>it</em> is the culprit.</p>

<p>TLDR: I think it might be safe to land it.</p>

<p>P.S. <a href="https://phabricator.kde.org/p/zzag/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@zzag</a> while I have had issues with corners, they've never looked like they do in your screenshots, are you sure you've applied <strong>both</strong> svgz (latest diff), removed the old ones and made sure that it's using the proper breeze-dark colors file (yours looks a little wan). Mine works like Nate's.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18545">https://phabricator.kde.org/D18545</a></div></div><br /><div><strong>To: </strong>rooty, VDG, ngraham<br /><strong>Cc: </strong>zzag, davidedmundson, Codezela, filipf, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>