Corrected chart width for sub-pixel container width. Prevented overfl…

…ow in flex-grids and table cells. See #6427. Closes #1758. Closes #5070. Closes #5153. Closes #5673. Closes #8312.
highcharts · Nov 5, 2018 · 02b559d · 02b559d
1 parent 8547ef0
commit 02b559d
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/js/parts/Chart.js b/js/parts/Chart.js
@@ -1134,6 +1134,10 @@ extend(Chart.prototype, /** @lends Highcharts.Chart.prototype */ {
         chartWidth = chart.chartWidth;
         chartHeight = chart.chartHeight;
 
+        // Allow table cells and flex-boxes to shrink without the chart blocking
+        // them out (#6427)
+        css(renderTo, { overflow: 'hidden' });
+
         // Create the inner container
         /*= if (build.classic) { =*/
         containerStyle = extend({

diff --git a/js/parts/Utilities.js b/js/parts/Utilities.js
@@ -2009,9 +2009,17 @@ H.getStyle = function (el, prop, toInt) {
     if (prop === 'width') {
         return Math.max(
             0, // #8377
-            Math.min(el.offsetWidth, el.scrollWidth) -
+            (
+                Math.min(
+                    el.offsetWidth,
+                    el.scrollWidth,
+                    el.getBoundingClientRect ?
+                        Math.floor(el.getBoundingClientRect().width) : // #6427
+                        Infinity
+                ) -
                 H.getStyle(el, 'padding-left') -
                 H.getStyle(el, 'padding-right')
+            )
         );
     } else if (prop === 'height') {
         return Math.max(