hpereiradacosta added a comment.
Hi, ,Thanks for the patch ! Do you have an example application for which this fixes the issue ? I'm a bit worried about the possible regressions that your change introduced, so I would like to understand better what happens here ... INLINE COMMENTS > breezestyle.cpp:3067 > + // no focus indicator on buttons / scrollbars, since it is rendered > elsewhere > + if ( qobject_cast< const QAbstractButton*>( widget ) || > qobject_cast< const QScrollBar*>( widget ) ) > return true; You sure this is actually needed ? The primitive is supposed to do an horizontal line below the passed rect. Never seen that happened for scrollbars. Does it really ? (without the patch) REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D3813 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca, #plasma, hpereiradacosta Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
