Fixed issue #4133, error bar dataLabels positions failing on redraw.

highcharts · Sep 9, 2016 · ff1e4d7 · ff1e4d7
1 parent 3b9c3af
commit ff1e4d7
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/js/highcharts-more.src.js b/js/highcharts-more.src.js
@@ -1622,7 +1622,14 @@ var arrayMin = Highcharts.arrayMin,
         },
         pointValKey: 'high', // defines the top of the tracker
         doQuartiles: false,
-        drawDataLabels: seriesTypes.arearange ? seriesTypes.arearange.prototype.drawDataLabels : noop,
+        drawDataLabels: seriesTypes.arearange ? function () {
+            var valKey = this.pointValKey;
+            seriesTypes.arearange.prototype.drawDataLabels.call(this);
+            // Arearange drawDataLabels does not reset point.y to high, but to low after drawing. #4133 
+            each(this.data, function (point) {
+                point.y = point[valKey];
+            });
+        } : noop,
 
         /**
          * Get the width and X offset, either on top of the linked series column

diff --git a/js/parts-more/ErrorBarSeries.js b/js/parts-more/ErrorBarSeries.js
@@ -22,7 +22,14 @@ seriesTypes.errorbar = extendClass(seriesTypes.boxplot, {
 	},
 	pointValKey: 'high', // defines the top of the tracker
 	doQuartiles: false,
-	drawDataLabels: seriesTypes.arearange ? seriesTypes.arearange.prototype.drawDataLabels : noop,
+	drawDataLabels: seriesTypes.arearange ? function () {
+		var valKey = this.pointValKey;
+		seriesTypes.arearange.prototype.drawDataLabels.call(this);
+		// Arearange drawDataLabels does not reset point.y to high, but to low after drawing. #4133 
+		each(this.data, function (point) {
+			point.y = point[valKey];
+		});
+	} : noop,
 
 	/**
 	 * Get the width and X offset, either on top of the linked series column