summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndras Timar <andras.timar@collabora.com>2015-04-04 14:13:18 +0200
committerAndras Timar <andras.timar@collabora.com>2015-04-04 17:12:49 +0000
commit6220362e844ad0e50a38c1627ddd05a082882009 (patch)
tree4a3c92c829f86138985f8082ca04cc319a4e1101
parent1a3c99f2c31dbc4dd6b3b5788ea0b49a34bea2c6 (diff)
tdf#90404 OOXML chart legend import improvement
In the bugdoc, the chart legend had 6 entries in PowerPoint, but only 5 in Impress. I played a lot with padding and offset parameters in layout algorithm without success. Finally I introduced this 1mm tolerance, so the fixed legend frame can expand up to 1mm, if necessary. Change-Id: Id1ad1df67aab41d942133cea8d16fccf0f215167 Reviewed-on: https://gerrit.libreoffice.org/15149 Reviewed-by: Andras Timar <andras.timar@collabora.com> Tested-by: Andras Timar <andras.timar@collabora.com>
-rw-r--r--chart2/source/view/main/VLegend.cxx4
1 files changed, 2 insertions, 2 deletions
diff --git a/chart2/source/view/main/VLegend.cxx b/chart2/source/view/main/VLegend.cxx
index a5a3b722a97e..d2931654f9d3 100644
--- a/chart2/source/view/main/VLegend.cxx
+++ b/chart2/source/view/main/VLegend.cxx
@@ -390,7 +390,7 @@ awt::Size lcl_placeLegendEntries(
nSumHeight += aRowHeights[nR];
sal_Int32 nRemainingSpace = rAvailableSpace.Height - nSumHeight;
- if( nRemainingSpace<0 )
+ if( nRemainingSpace < -100 ) // 1mm tolerance for OOXML interop tdf#90404
{
//remove entries that are too big
for( sal_Int32 nR=nNumberOfRows; nR--; )
@@ -418,7 +418,7 @@ awt::Size lcl_placeLegendEntries(
}
nNumberOfRows = static_cast<sal_Int32>(aRowHeights.size());
}
- if( nRemainingSpace > 0 )
+ if( nRemainingSpace >= -100 ) // 1mm tolerance for OOXML interop tdf#90404
{
sal_Int32 nNormalSpacingHeight = 2*nYPadding+(nNumberOfRows-1)*nYOffset;
if( nRemainingSpace < nNormalSpacingHeight )