fix(resize) : fix resizing bug with bootstrap and angular material

fix pablojim#550
houssemba · Mar 2, 2017 · 1fd3463 · 1fd3463
1 parent bf24c5b
commit 1fd3463
Show file tree

Hide file tree

Showing 5 changed files with 33 additions and 3 deletions.
diff --git a/dist/highcharts-ng.css b/dist/highcharts-ng.css
@@ -0,0 +1,5 @@
+highcharts {
+  display: block;
+  width: 100%;
+  max-width: 100%;
+}
diff --git a/dist/highcharts-ng.js b/dist/highcharts-ng.js
@@ -1,6 +1,6 @@
 /**
  * highcharts-ng
- * @version v1.0.2-dev - 2017-02-20
+ * @version v1.0.2-dev - 2017-03-02
  * @link https://github.com/pablojim/highcharts-ng
  * @author Barry Fitzgerald <>
  * @license MIT License, http://www.opensource.org/licenses/MIT
@@ -39,6 +39,16 @@ if (typeof module !== 'undefined' && typeof exports !== 'undefined' && module.ex
       prevConfig = angular.merge({}, ctrl.config);
       mergedConfig = getMergedOptions($element, ctrl.config, seriesId);
       ctrl.chart = new Highcharts[getChartType(mergedConfig)](mergedConfig);
+
+      // Fix resizing bug
+      // https://github.com/pablojim/highcharts-ng/issues/550
+      var originalWidth = $element[0].clientWidth;
+      var originalHeight = $element[0].clientHeight;
+      $timeout(function() {
+        if ($element[0].clientWidth !== originalWidth || $element[0].clientHeight !== originalHeight) {
+          ctrl.chart.reflow();
+        }
+      }, 0, false);
     };
 
     this.$doCheck = function() {

diff --git a/dist/highcharts-ng.min.js b/dist/highcharts-ng.min.js
diff --git a/src/highcharts-ng.css b/src/highcharts-ng.css
@@ -0,0 +1,5 @@
+highcharts {
+  display: block;
+  width: 100%;
+  max-width: 100%;
+}
diff --git a/src/highcharts-ng.js b/src/highcharts-ng.js
@@ -31,6 +31,16 @@ if (typeof module !== 'undefined' && typeof exports !== 'undefined' && module.ex
       prevConfig = angular.merge({}, ctrl.config);
       mergedConfig = getMergedOptions($element, ctrl.config, seriesId);
       ctrl.chart = new Highcharts[getChartType(mergedConfig)](mergedConfig);
+
+      // Fix resizing bug
+      // https://github.com/pablojim/highcharts-ng/issues/550
+      var originalWidth = $element[0].clientWidth;
+      var originalHeight = $element[0].clientHeight;
+      $timeout(function() {
+        if ($element[0].clientWidth !== originalWidth || $element[0].clientHeight !== originalHeight) {
+          ctrl.chart.reflow();
+        }
+      }, 0, false);
     };
 
     this.$doCheck = function() {