Highstock: Fixed highcharts#9693, added default groupPixelWidth when …

…dataGrouping options were set to all series through plotOptions.series.
Jessica7 · Jan 3, 2019 · 8b7eb06 · 8b7eb06
1 parent f0e56e3
commit 8b7eb06
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/js/parts/DataGrouping.js b/js/parts/DataGrouping.js
@@ -964,7 +964,7 @@ Axis.prototype.getGroupPixelWidth = function () {
         if (dgOptions) {
             groupPixelWidth = Math.max(
                 groupPixelWidth,
-                dgOptions.groupPixelWidth
+                pick(dgOptions.groupPixelWidth, 2) // #9693
             );
 
         }

diff --git a/samples/unit-tests/series/datagrouping/demo.js b/samples/unit-tests/series/datagrouping/demo.js
@@ -34,9 +34,19 @@ QUnit.test('General dataGrouping options', function (assert) {
             data: [
                 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1
             ]
+        }, {
+            type: 'scatter',
+            data: [[1, 1]]
         }]
     });
 
+    assert.strictEqual(
+        chart.series[0].points[0].y > 1,
+        true,
+        'Scatter doesn\'t prevent dataGrouping when ' +
+            '`plotOptions.series.dataGrouping` is set (#9693)'
+    );
+
     assert.strictEqual(
         chart.series[0].points[0].y,
         5,