Hi Joel,
fdo: #47164:
it should be fdo#47164, otherwise the commit would not be recognized as
a bug-fix.
Please enter the commit message for your changes. Lines starting
> ...
How did you manage to get this lines in the commit message? :) Please
remove those.
- pDev->DrawRect( Rectangle( nX+0, nY+0, nX+6, nY+0 ) );
+ pDev->DrawRect( Rectangle( nX-0, nY+0, nX+6, nY+0 ) );
By the way, such changes make reviewing harder.
+ if( ! bLayout )
+ {
+ long nDistance = (aInRect.GetHeight() > 10) ? 2 : 1;
+ DecorationView aDecoView( pDev );
+ if( bMenuBtnSep )
+ {
+ long nX = aInRect.Left() - 3*nDistance;
+ }
+ aDecoView.DrawSymbol( aInRect, SYMBOL_SPIN_DOWN, aColor, nStyle );
+ }
nX is unused (and thus nDistance is unused as well). What was the
intension here?
@@ -880,13 +900,12 @@ void PushButton::ImplDrawPushButtonContent( OutputDevice*
pDev, sal_uLong nDrawF
DecorationView aDecoView( pDev );
if( bMenuBtnSep )
{
- long nX = aInRect.Left() - 2*nDistance;
- Point aStartPt( nX, aInRect.Top()+nDistance );
+ long nX = aInRect.Left() - 3*nDistance;
+ Point aStartPt( nX+10, aInRect.Top()+nDistance );
Point aEndPt( nX, aInRect.Bottom()-nDistance );
aDecoView.DrawSeparator( aStartPt, aEndPt );
}
the x coordinate is different in the start point and the end point. You
can see it, run:
$ export SAL_USE_VCLPLUGIN=gen && ./install/program/soffice
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.
Could you send an updated patch?
Thanks,
Ivan
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
