Sure thing. I'll try to fix it tonight. As for the magic numbers, I agree, the original way it was done just wasn't good in my opinion and made fixing it hard and partly redundant. It might take me an extra day or two because I might try to make it a bit cleaner than it currently is. The original code linked the size of the arrow with the hot spot which meant to increase the size of the hotspot you automatically increased the size of the arrow.<div>
<br></div><div>As for nX, this was simply copied from the original code. I'll relook at it but for some reason I remember getting an error when I removed it. </div><div><br></div><div>Give me a day or two. </div><div>
<br></div><div>Joel<br><div><br></div><div><br><div><br><div class="gmail_quote">On Tue, Jul 24, 2012 at 9:23 AM, Ivan Timofeev <span dir="ltr"><<a href="mailto:timofeev.i.s@gmail.com" target="_blank">timofeev.i.s@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Joel,<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
fdo: #47164:<br>
</blockquote>
<br>
it should be fdo#47164, otherwise the commit would not be recognized as a bug-fix.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 Please enter the commit message for your changes. Lines starting<br>
</blockquote>
> ...<br>
<br>
How did you manage to get this lines in the commit message? :) Please remove those.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-    pDev->DrawRect( Rectangle( nX+0, nY+0, nX+6, nY+0 ) );<br>
+    pDev->DrawRect( Rectangle( nX-0, nY+0, nX+6, nY+0 ) );<br>
</blockquote>
<br>
By the way, such changes make reviewing harder.<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+        if( ! bLayout )<br>
+        {<br>
+            long nDistance = (aInRect.GetHeight() > 10) ? 2 : 1;<br>
+            DecorationView aDecoView( pDev );<br>
+            if( bMenuBtnSep )<br>
+            {<br>
+                long nX = aInRect.Left() - 3*nDistance;<br>
+            }<br>
<br>
+            aDecoView.DrawSymbol( aInRect, SYMBOL_SPIN_DOWN, aColor, nStyle );<br>
+        }<br>
</blockquote>
<br>
nX is unused (and thus nDistance is unused as well). What was the intension here?<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
@@ -880,13 +900,12 @@ void PushButton::<u></u>ImplDrawPushButtonContent( OutputDevice* pDev, sal_uLong nDrawF<br>
             DecorationView aDecoView( pDev );<br>
             if( bMenuBtnSep )<br>
             {<br>
-                long nX = aInRect.Left() - 2*nDistance;<br>
-                Point aStartPt( nX, aInRect.Top()+nDistance );<br>
+                long nX = aInRect.Left() - 3*nDistance;<br>
+                Point aStartPt( nX+10, aInRect.Top()+nDistance );<br>
                 Point aEndPt( nX, aInRect.Bottom()-nDistance );<br>
                 aDecoView.DrawSeparator( aStartPt, aEndPt );<br>
             }<br>
</blockquote>
<br>
the x coordinate is different in the start point and the end point. You can see it, run:<br>
<br>
 $ export SAL_USE_VCLPLUGIN=gen && ./install/program/soffice<br>
<br>
<br>
I have absolutely no idea of those magic numbers that was there in button.cxx, but it seems you adjusted them right, the patch in general works fine.<br>
<br>
Could you send an updated patch?<br>
<br>
Thanks,<br>
<br>
Ivan<br>
</blockquote></div><br></div></div></div>