Revert "[DE, PDFE, PE] fix: reset data labels state before setting position"

This reverts commit 122095eb64.
This commit is contained in:
Dmitry-Ilyushechkin
2025-10-08 13:08:36 +03:00
parent 122095eb64
commit f8068259ac
3 changed files with 2 additions and 29 deletions

View File

@ -956,39 +956,30 @@ define([], function () {
chartProps.setDisplayChartTitle(true, true);
break;
case 'CenterData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.ctr);
break;
case 'InnerBottomData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.inBase);
break;
case 'InnerTopData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.inEnd);
break;
case 'OuterTopData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.outEnd);
break;
case 'TopData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.t);
break;
case 'LeftData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.l);
break;
case 'RightData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.r);
break;
case 'BottomData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.b);
break;
case 'FitWidthData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.bestFit);
break;
case 'bShowDataLabels':

View File

@ -2782,39 +2782,30 @@ define([], function () {
chartProps.setDisplayChartTitle(true, true);
break;
case 'CenterData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.ctr);
break;
case 'InnerBottomData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.inBase);
break;
case 'InnerTopData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.inEnd);
break;
case 'OuterTopData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.outEnd);
break;
case 'TopData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.t);
break;
case 'LeftData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.l);
break;
case 'RightData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.r);
break;
case 'BottomData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.b);
break;
case 'FitWidthData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.bestFit);
break;
case 'bShowDataLabels':
@ -3067,7 +3058,7 @@ define([], function () {
me.isRtlSheet = me.api ? Common.UI.isRTL() : false;
if (me.chartProps) {
if (!me.btnChartElement) {
me.btnChartElement = new Common.UI.Button({
parentEl: $('#id-document-holder-btn-chart-element'),

View File

@ -911,39 +911,30 @@ define([], function () {
chartProps.setDisplayChartTitle(true, true);
break;
case 'CenterData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.ctr);
break;
case 'InnerBottomData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.inBase);
break;
case 'InnerTopData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.inEnd);
break;
case 'OuterTopData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.outEnd);
break;
case 'TopData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.t);
break;
case 'LeftData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.l);
break;
case 'RightData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.r);
break;
case 'BottomData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.b);
break;
case 'FitWidthData':
if (chartProps.getDataLabelsPos() === 0) chartProps.setDisplayDataLabels(false, false);
chartProps.setDisplayDataLabels(true, Asc.c_oAscChartDataLabelsPos.bestFit);
break;
case 'bShowDataLabels':
@ -1210,7 +1201,7 @@ define([], function () {
me.isRtlSheet = me.api ? Common.UI.isRTL() : false;
if (me.chartProps) {
if (!me.btnChartElement) {
me.btnChartElement = new Common.UI.Button({
parentEl: $('#id-document-holder-btn-chart-element'),