ndavis added a comment.
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. 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