<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>Seems my reply per email went AWOL:</p>

<p>This is a work-in-progress ticket, but I can change the title because it is indeed not just about reverting a regression.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>That just doesn't look good, sorry.</p></blockquote>

<p>Again, that's an argument one should avoid. It's too subjective. I myself find it looks much better (I'm biased against everything Breeze, sorry) and easier on the eyes because I prefer almost monochromatic palettes.</p>

<p>Yes, I said that the current implementation looks too gaudy and that's subjective too. What I think I *can* claim is that using a background colour that comes from the theme's own background colours makes message widgets fit much better within the theme's design. And of course it avoids violating a basic, common-sensical and straightforward instruction from KColorScheme: don't use foreground colours for background purposes. See also my remark below about impaired colour vision.</p>

<p>I have no readability problems with Breeze Dark (not any more than with dark themes in general). It should be possible to lighten or darken the text colour depending on contrast, to achieve a stronger contrast. Not very easy though due to (perceived) colour contrast effects. For instance:</p>

<p><a href="https://phabricator.kde.org/F6006588" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6006588: image.png</a></p>

<p>This does remind me of the fact that I have normal colour vision but a significant part of the population has impaired colour vision. It's tricky to start mixing colours like the original code does, and guarantee that the resulting colour pair can be distinguished sufficiently well by everyone. Simple brightening/darkening should be safer but not 100% so either, probably. In short, themes designed for specific visual shortcomings should continue to work better if you only use colours from those themes, and for their intended purposes.</p>

<p>Re: splitting:<br />
If the KThemeSettings class looks more or less like what you had in mind and something that can go in quickly I'll split that off and then we can take things from there.</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>