antlarr added inline comments.

INLINE COMMENTS

> davidedmundson wrote in breezestyle.cpp:2507
> you're still basing the height off the MenuButton_IndicatorWidth which you're 
> otherwise not using
> 
> Is that intentional?

Good point, I didn't notice that it was using MenuButton_IndicatorWidth  to set 
the height, but still, height seems to be fine, so I wouldn't change that.

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