davidedmundson added inline comments.

INLINE COMMENTS

> breezestyle.cpp:2507
>          // make sure there is enough height for the button
>          size.setHeight( qMax( size.height(), 
> int(Metrics::MenuButton_IndicatorWidth) ) );
>  

you're still basing the height off the MenuButton_IndicatorWidth which you're 
otherwise not using

Is that intentional?

REPOSITORY
  rBREEZE Breeze

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: antlarr, #plasma, #plasma:_design
Cc: davidedmundson, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, htietze, sebas

Reply via email to