ngraham added a comment.
I see what you're trying to do here, and I approve. But I think it might make more sense to more generally rethink how we indicate focus for buttons. IMO we should draw a highlight around the edges rather than changing the background color, for the following reasons: - It would avoid any confusion whatsoever regarding checkable buttons - It would make the currently focused button stop inappropriately looking like it's a dialog's default button - It would be more consistent with how focus is indicated for other UI elements in Breeze (e.g toolbuttons and line edits) - It would be more consistent with how other focus is indicated for other widget styles and platforms (e.g. macOS, Windows, and GNOME's Adwaits all show focus with a line around the edges) REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D24706 To: ndavis, #vdg, #breeze Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
