<table><tr><td style="">ndavis 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/D27263">View Revision</a></tr></table><br /><div><div><p>I know this isn't the final version, but I'm not a fan of this idea. It feels like we'd just piling new stuff on top without properly using the old stuff that is already usable. I think if we're not careful, it'll just make color customization more confusing and difficult to work with by causing unexpected behavior where the old and new systems collide and one overrides the other.</p>

<p>Do we really need to create a color setting for accent that is separate from QPalette::Highlight/Selection Background? If you really want accent colors, I'd say deprecate FocusColor and HoverColor. As a fallback, have focus and hover state colors be based on QPalette::Highlight/Selection Background and set QPalette::Highlight directly instead of defining a separate accent color.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R265 KConfigWidgets</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27263">https://phabricator.kde.org/D27263</a></div></div><br /><div><strong>To: </strong>cblack, Plasma<br /><strong>Cc: </strong>The-Feren-OS-Dev, ngraham, kde-frameworks-devel, ndavis, LeGast00n, cblack, GB_2, michaelh, bruns<br /></div>