<table><tr><td style="">rjvbb updated this revision to Diff 36886.<br />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>A new revision:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">makes a start with the requested export of the QSettings functionality</li>
<li class="remarkup-list-item">drops the alpha tweak and introduces a new experiment:</li>
</ul>

<p>Since background colours are not obtained from theme colours designed for that purpose AND they're mixed with the widget background, I tried to use the hardcoded default colours as a reference. Instead of changing the selected theme colours in a qualitative fashion they are brightness-corrected to give them the same perceived intensity as the default colours as much as that is possible in RGB space. Basically this is greyscale normalisation done individually for each colour role, and the idea is to achieve a more consistent contrast pattern across message roles and themes.</p>

<p>The correction is activated by an env. variable (<tt style="background: #ebebeb; font-size: 13px;">KMESSAGEWIDGET_BACKGROUND_BRIGHTNESS_MATCHED</tt>). I introduced to make testing comparisons easier (= avoid recompilation). The effect is very subtle with the Breeze themes (improves readability a bit for the dark ones, I think), stronger with my own theme. It may however have unexpected effects so maybe the switch should be preserved IF this experiment is maintained.</p></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D13777?vs=36858&id=36886">https://phabricator.kde.org/D13777?vs=36858&id=36886</a></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>AFFECTED FILES</strong><div><div>src/kmessagewidget.cpp</div></div></div><br /><div><strong>To: </strong>rjvbb, ngraham, Frameworks<br /><strong>Cc: </strong>cfeck, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>