[Differential] [Changed Subscribers] D3192: The combobox needs to be 2 pixels wider for contents to fit

[davidedmundson (David Edmundson)]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161031/59e480f5/attachment.html>