hpereiradacosta added inline comments.

INLINE COMMENTS

> breezestyle.cpp:407
> +                pal.setColor( QPalette::Active, QPalette::Base, 
> scrollArea->palette().color( scrollArea->backgroundRole() ) );
> +                scrollArea->setPalette(pal);
>              }

Setting palettes manually is always somewhat dangerous, (can break child 
widgets for instance, which would inherit this palette). 
Can we at least hide this code behind a check for QTreeView, and even 
"isAnimated()" ?

REPOSITORY
  R31 Breeze

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

To: broulik, #plasma, hpereiradacosta
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart, lukas

Reply via email to