Fixed #2597, data labels remained visible even though container was h…

…idden.
highcharts · Jul 27, 2016 · 981fd78 · 981fd78
1 parent 5467267
commit 981fd78
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/js/highcharts.src.js b/js/highcharts.src.js
@@ -2,7 +2,7 @@
 // @compilation_level SIMPLE_OPTIMIZATIONS
 
 /**
- * @license Highcharts JS v4.2.5-modified (2016-07-26)
+ * @license Highcharts JS v4.2.5-modified (2016-07-27)
  *
  * (c) 2009-2016 Torstein Honsi
  *
@@ -17993,8 +17993,8 @@
                 dataLabelsGroup.attr({ opacity: +hasRendered }); // #3300
                 if (!hasRendered) {
                     addEvent(series, 'afterAnimate', function () {
-                        if (series.visible) { // #3023, #3024
-                            dataLabelsGroup.show();
+                        if (series.visible) { // #2597, #3023, #3024
+                            dataLabelsGroup.show(true);
                         }
                         dataLabelsGroup[seriesOptions.animation ? 'animate' : 'attr']({ opacity: 1 }, { duration: 200 });
                     });

diff --git a/js/highmaps.src.js b/js/highmaps.src.js
@@ -1,5 +1,5 @@
 /**
- * @license Highmaps JS v4.2.5-modified (2016-07-26)
+ * @license Highmaps JS v4.2.5-modified (2016-07-27)
  *
  * (c) 2011-2016 Torstein Honsi
  *
@@ -16143,8 +16143,8 @@
                 dataLabelsGroup.attr({ opacity: +hasRendered }); // #3300
                 if (!hasRendered) {
                     addEvent(series, 'afterAnimate', function () {
-                        if (series.visible) { // #3023, #3024
-                            dataLabelsGroup.show();
+                        if (series.visible) { // #2597, #3023, #3024
+                            dataLabelsGroup.show(true);
                         }
                         dataLabelsGroup[seriesOptions.animation ? 'animate' : 'attr']({ opacity: 1 }, { duration: 200 });
                     });

diff --git a/js/highstock.src.js b/js/highstock.src.js
@@ -17993,8 +17993,8 @@
                 dataLabelsGroup.attr({ opacity: +hasRendered }); // #3300
                 if (!hasRendered) {
                     addEvent(series, 'afterAnimate', function () {
-                        if (series.visible) { // #3023, #3024
-                            dataLabelsGroup.show();
+                        if (series.visible) { // #2597, #3023, #3024
+                            dataLabelsGroup.show(true);
                         }
                         dataLabelsGroup[seriesOptions.animation ? 'animate' : 'attr']({ opacity: 1 }, { duration: 200 });
                     });

diff --git a/js/parts/DataLabels.js b/js/parts/DataLabels.js
@@ -35,8 +35,8 @@ Series.prototype.drawDataLabels = function () {
 			dataLabelsGroup.attr({ opacity: +hasRendered }); // #3300
 			if (!hasRendered) {
 				addEvent(series, 'afterAnimate', function () {
-					if (series.visible) { // #3023, #3024
-						dataLabelsGroup.show();
+					if (series.visible) { // #2597, #3023, #3024
+						dataLabelsGroup.show(true);
 					}
 					dataLabelsGroup[seriesOptions.animation ? 'animate' : 'attr']({ opacity: 1 }, { duration: 200 });
 				});