<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/D13777">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/D13777#286439" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D13777#286439</a>, <a href="https://phabricator.kde.org/p/rjvbb/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@rjvbb</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Have you guys considered using the 4 colours in question only for the message text and outer frame, keeping the background intact (or possibly just a bit lighter or darker depending on its initial brightness and/or the theme/user's contrast setting)? The result should be more subtle (edit: less gaudy) and more reliably as readable as allowed by the theme.<br />
 [...]<br />
 <a href="https://phabricator.kde.org/F6005564" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6005564: image.png</a></p>

<p>That's Breeze vs. Breeze Dark vs. my custom theme.</p></div>
</blockquote>

<p>I'm against this purely on visual grounds. That just doesn't look good, sorry. It also makes the link text very hard to read with Breeze dark, regressing something we were explicitly trying to fix with the current design.</p>

<p>I don't see how this patch can move forward as long as it's a mix of uncontroversial changes (use colors from the theme) and huge conceptual changes (totally change how colors are used and the color and alpha of the background). Allow me to repeat my request to split this up into multiple patches so that we can move forward with something. If you need some help juggling multiple patches using arc, I'd be happy to lend a hand.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R236 KWidgetsAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13777">https://phabricator.kde.org/D13777</a></div></div><br /><div><strong>To: </strong>rjvbb, ngraham, Frameworks, VDG<br /><strong>Cc: </strong>aacid, cfeck, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>