D24706: Don't use focus color on checkable button backgrounds

[Nathaniel Graham]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191016/387e428f/attachment-0001.html>