oox/source/drawingml/chart/seriesconverter.cxx | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
New commits:
commit 42fd10b0ab6c6f65ba6394f9ae216c0f13973221
Author: Balazs Varga <balazs.varga...@gmail.com>
AuthorDate: Mon Feb 18 12:50:03 2019 +0100
Commit: László Németh <nem...@numbertext.org>
CommitDate: Wed Feb 20 10:47:16 2019 +0100
Related: tdf#122226 OOXML Chart Import data label separator
Set the data label separator to "new line" if there is not
present explicit point separator and the "percentage format
wins over number value format". In MS-Office (2007/2010/2013/2016)
the defult separator is the "new line" in that case, any other
case the separator is a semicolon.
Change-Id: I7f562d814b74eba15b2fba2954efd42747dd7960
Reviewed-on: https://gerrit.libreoffice.org/67974
Tested-by: Jenkins
Reviewed-by: László Németh <nem...@numbertext.org>
diff --git a/oox/source/drawingml/chart/seriesconverter.cxx
b/oox/source/drawingml/chart/seriesconverter.cxx
index 8fd572d98faf..01e1d72562c5 100644
--- a/oox/source/drawingml/chart/seriesconverter.cxx
+++ b/oox/source/drawingml/chart/seriesconverter.cxx
@@ -182,8 +182,12 @@ void lclConvertLabelFormatting( PropertySet& rPropSet,
ObjectFormatter& rFormatt
convertTextProperty(rPropSet, rFormatter, rDataLabel.mxTextProp);
// data label separator (do not overwrite series separator, if no
explicit point separator is present)
- if( bDataSeriesLabel || rDataLabel.moaSeparator.has() )
+ // Set the data label separator to "new line" if the value is shown as
percentage with a category name,
+ // just like in MS-Office. In any other case the default separator
will be a semicolon.
+ if( bShowPercent && !bShowValue && ( bDataSeriesLabel ||
rDataLabel.moaSeparator.has() ) )
rPropSet.setProperty( PROP_LabelSeparator,
rDataLabel.moaSeparator.get( "\n" ) );
+ else if( bDataSeriesLabel || rDataLabel.moaSeparator.has() )
+ rPropSet.setProperty( PROP_LabelSeparator,
rDataLabel.moaSeparator.get( "; " ) );
// data label placement (do not overwrite series placement, if no
explicit point placement is present)
if( bDataSeriesLabel || rDataLabel.monLabelPos.has() )
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
