<table><tr><td style="">ngraham edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-yhrx75cfjjd4oee/">(Show Details)</a>
</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/D12756">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">When I started work on this, I first wrote</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Use</span> a conditional lightness check <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">that switched between `Qt::red` and `Qt::darkRed` depending on th</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">to determin</span>e th<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">eme's window background</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">e</span> color. <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">That worked</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">For now</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">but then I noticed that</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">use</span> `Qt::<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">darkR</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">r</span>ed` i<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">sn't all that visible for Breeze light in the first place.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">nstead of `Qt::darkRed`,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">`Qt::red` looks better and is more visible for both of them</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">but really</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">so let's just use that instead!</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">we should use a better set of colors..</span><br />
<br />
BUG: 389075<div style="padding: 8px 0;">...</div></div></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/D12756">https://phabricator.kde.org/D12756</a></div></div><br /><div><strong>To: </strong>ngraham, Frameworks<br /><strong>Cc: </strong>cfeck, kde-frameworks-devel, mwolff, apol, michaelh, ngraham, bruns<br /></div>