<table><tr><td style="">rjvbb updated this revision to Diff 36939.<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>Using the theme's normal foreground colour for message text for more reliable readability with darker themes.</p>

<p>Brightness matching to the fallback colour should never occur here, at least not when the palette text colour is black (as it tends to be in my testing). Brightness matching would make the resulting colour just about as dark (matching to a darker colour almost always succeeds).</p></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D13777?vs=36886&id=36939">https://phabricator.kde.org/D13777?vs=36886&id=36939</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>