summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNoel Power <noel.power@novell.com>2012-03-27 12:32:05 +0100
committerNoel Power <noel.power@novell.com>2012-03-27 12:38:49 +0100
commit5ae64e4b0c23f209410fe84df041c9445234df74 (patch)
tree35c380056d1c31413b7dbecd38d0564d16f29923
parentdd2fe95cce75f1157bd1c75d286a0047b2e4175e (diff)
fix vertical align when switching mode in multibar fdo#44140
switching between SumAssign & OkCancel mode in the multibar results in a slight 'jumping' of the toolbar items when end up out of allignment with the other toolbar items.
-rw-r--r--vcl/source/window/toolbox.cxx14
1 files changed, 13 insertions, 1 deletions
diff --git a/vcl/source/window/toolbox.cxx b/vcl/source/window/toolbox.cxx
index 4dc2458acc58..7b317a180651 100644
--- a/vcl/source/window/toolbox.cxx
+++ b/vcl/source/window/toolbox.cxx
@@ -2722,7 +2722,19 @@ void ToolBox::ImplFormat( sal_Bool bResize )
// if special TBX_LAYOUT_LOCKVERT lock vertical position
// don't recalulate the vertical position of the item
if ( meLayoutMode == TBX_LAYOUT_LOCKVERT && mnLines == 1 )
- it->maCalcRect.Top() = it->maRect.Top();
+ {
+ // Somewhat of a hack here, calc deletes and re-adds
+ // the sum/assign & ok/cancel items dynamically.
+ // Because TBX_LAYOUT_LOCKVERT effectively prevents
+ // recalculation of the vertical pos of an item the
+ // it->maRect.Top() for those newly added items is
+ // 0. The hack here is that we want to effectively
+ // recalculate the vertical pos for those added
+ // items here. ( Note: assume mnMaxItemHeight is
+ // equal to the LineSize when multibar has a single
+ // line size )
+ it->maCalcRect.Top() = it->maRect.Top() ? it->maRect.Top() : ( nY + ( mnMaxItemHeight-aCurrentItemSize.Height())/2 );
+ }
else
it->maCalcRect.Top() = nY+(nLineSize-aCurrentItemSize.Height())/2;
it->maCalcRect.Right() = nX+aCurrentItemSize.Width()-1;