ndavis added a comment.
In D24706#552981 <https://phabricator.kde.org/D24706#552981>, @ndavis wrote: > In D24706#552980 <https://phabricator.kde.org/D24706#552980>, @ngraham wrote: > > > Combobox still affected: F7659165: Screenshot_20191023_145546.png <https://phabricator.kde.org/F7659165> > > > > Icons on focused buttons (ToolButtons as well as PushButtons) still have reversed colors: F7659169: Screenshot_20191023_145628.png <https://phabricator.kde.org/F7659169> > > > > Looks like the focused button background's color still changes a bit, too. In Breeze non-dark, it becomes a lighter shade of gray. I thought we were going to stop changing the background color for focused buttons and on;y use outlines? > > > It has a different BG color because it's a default button, not just because it's focused. That has nothing to do with the patch (yet). I'll try to figure out why the icons are screwing up. Actually, I might be wrong. For some reason, the focused background is still colored differently. I don't know where that's coming from. REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D24706 To: ndavis, #vdg, #breeze Cc: bodoeggert, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, cblack, konkinartem, ian, jguidon, hannahk, Ghost6, jraleigh, MrPepe, fbampaloukas, squeakypancakes, alexde, IohannesPetros, GB_2, trickyricky26, ragreen, mglb, crozbo, ndavis, ZrenBot, firef, alexeymin, skadinna, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, aaronhoneycutt, abetts, sebas, apol, ahiemstra, mbohlender, mart