<table><tr><td style="">ndavis created this revision.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />ndavis requested review of this revision.
</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/D26745">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Buttons colored with special text role colors as backgrounds don't always look nice with black text on light themes. They should have been using background role colors for the background, but the background role colors they needed weren't available. This patch adds the missing background role colors.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R858 Qt Quick Controls 2: Desktop Style</div></div></div><br /><div><strong>BRANCH</strong><div><div>more-background-colors (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26745">https://phabricator.kde.org/D26745</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kirigami-plasmadesktop-integration/plasmadesktoptheme.cpp</div></div></div><br /><div><strong>To: </strong>ndavis<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>