D9227: [Breeze widget style] Override QLineEditIconButton paint event to force Active icon on press

[Hugo Pereira Da Costa]

hpereiradacosta added a comment.


  Hi Kai,
  Thanks for the patch.
  Just to make sure: can't we just manually modify the toolbuttons (well qlineediticonbutton) once and for all in say "::polish", so that the selected state == active state ? this would make the whole thing more robust I guess, rather than duplicating the paintEvent routine. 
  If not, no bother. I'll just review this code.

REPOSITORY
  R31 Breeze

REVISION DETAIL
  https://phabricator.kde.org/D9227

To: broulik, #plasma, #vdg, hpereiradacosta
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171206/d94b6dbf/attachment.html>