<table><tr><td style="">rjvbb 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/D13777">View Revision</a></tr></table><br /><div><div><p>Since there's been mention about aligning Kirigami and re: the (IMHO) controversial approach of using foreground (text) colours for background purposes:</p>

<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 and more reliably as readable as allowed by the theme.</p>

<p>I've done a very quick assessment, using the 0.2 alpha to darken dark window backgrounds and brighten light window backgrounds (using QColor::darker and QColor::lighter) which gives</p>

<p><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>

<p>two observations:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">lighter background colours are brightened less progressively to avoid hitting pure white too quickly</li>
<li class="remarkup-list-item">a hair-thin black outline for the text could increase readability for some combinations but appears to be impossible with stylesheets. (This is true in general, not just for this experiment.)</li>
</ul></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<br /><strong>Cc: </strong>aacid, cfeck, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>