<div dir="ltr">On Mon, Aug 26, 2013 at 12:48 AM, Milian Wolff <span dir="ltr"><<a href="mailto:mail@milianw.de" target="_blank">mail@milianw.de</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Monday 26 August 2013 00:36:34 Aleix Pol wrote:<br>
> Well... I actually tested it and didn't. I don't really mind but you could<br>
> have said that on the review...<br>
<br>
</div>Sorry, I didn't assume you tested it with QToolButton and thought it's simpler<br>
if I try it out myself and see whether it looks better or not.<br>
<br>
Why did you prefer the QPushButton variant? Personally, I prefer the<br>
QToolButton as it is not as cramped as the QPushButton variant, the margins<br>
look better.<br>
<br>
Should I show before/after screenshots?<br>
<br>
Cheers<br>
<span class="HOEnZb"><font color="#888888">--<br>
Milian Wolff<br>
<a href="mailto:mail@milianw.de">mail@milianw.de</a><br>
<a href="http://milianw.de" target="_blank">http://milianw.de</a><br>
</font></span></blockquote></div><br></div><div class="gmail_extra">I was getting a flat button, which integrates better with the menu (see review screenshot), although I realize I didn't leave it explicitly (I had some battling done to make sure there's no gray space between the menu and the toolbar anymore).</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">Aleix</div></div>