highcharts 7.1.1 → 7.1.2

package/es-modules/annotations/types/Measure.js

@@ -22,9 +22,7 @@ H.extendAnnotation(Measure, null,
     /** @lends Annotation.Measure# */
     {
         /**
-         *
          * Init annotation object.
-         *
          */
         init: function () {
             Annotation.prototype.init.apply(this, arguments);
@@ -123,8 +121,7 @@ H.extendAnnotation(Measure, null,
         /**
          * Add label with calculated values (min, max, average, bins).
          *
-         * @param {Boolean} resize - the flag for resize shape
-         *
+         * @param {boolean} resize - the flag for resize shape
          */
         addValues: function (resize) {
             var options = this.options.typeOptions,
@@ -175,8 +172,7 @@ H.extendAnnotation(Measure, null,
             }
         },
         /**
-         * add shapes - crosshair, background (rect)
-         *
+         * Crosshair, background (rect)
          */
         addShapes: function () {
             this.addCrosshairs();
@@ -375,9 +371,10 @@ H.extendAnnotation(Measure, null,
             this.options.typeOptions.point.y = this.startYMin;
         },
         calculations: {
-            /*
-            * Set starting points
-            */
+            /**
+             * Set starting points
+             * @private
+             */
             init: function () {
                 var options = this.options.typeOptions,
                     chart = this.chart,
@@ -425,12 +422,12 @@ H.extendAnnotation(Measure, null,
                 }
 
             },
-            /*
-            * Set current xAxisMin, xAxisMax, yAxisMin, yAxisMax.
-            * Calculations of measure values (min, max, average, bins).
-            *
-            * @param {Boolean} resize - flag if shape is resized
-            */
+            /**
+             * Set current xAxisMin, xAxisMax, yAxisMin, yAxisMax.
+             * Calculations of measure values (min, max, average, bins).
+             * @private
+             * @param {Boolean} resize - flag if shape is resized
+             */
             recalculate: function (resize) {
                 var calc = this.calculations,
                     options = this.options.typeOptions,
@@ -455,27 +452,27 @@ H.extendAnnotation(Measure, null,
                 }
 
             },
-            /*
-            * Set current xAxisMin, xAxisMax, yAxisMin, yAxisMax.
-            * Calculations of measure values (min, max, average, bins).
-            *
-            * @param {Object} axis - x or y axis reference
-            * @param {Number} value - point's value (x or y)
-            * @param {Number} offset - amount of pixels
-            */
+            /**
+             * Set current xAxisMin, xAxisMax, yAxisMin, yAxisMax.
+             * Calculations of measure values (min, max, average, bins).
+             * @private
+             * @param {Object} axis - x or y axis reference
+             * @param {Number} value - point's value (x or y)
+             * @param {Number} offset - amount of pixels
+             */
             getPointPos: function (axis, value, offset) {
                 return axis.toValue(
                     axis.toPixels(value) + offset
                 );
             },
-            /*
-            * Update position of start points
-            * (startXMin, startXMax, startYMin, startYMax)
-            *
-            * @param {Boolean} redraw - flag if shape is redraw
-            * @param {Boolean} resize - flag if shape is resized
-            * @param {Boolean} cpIndex - index of controlPoint
-            */
+            /**
+             * Update position of start points
+             * (startXMin, startXMax, startYMin, startYMax)
+             * @private
+             * @param {Boolean} redraw - flag if shape is redraw
+             * @param {Boolean} resize - flag if shape is resized
+             * @param {Boolean} cpIndex - index of controlPoint
+             */
             updateStartPoints: function (redraw, resize, cpIndex, dx, dy) {
                 var options = this.options.typeOptions,
                     selectType = options.selectType,
@@ -518,19 +515,21 @@ H.extendAnnotation(Measure, null,
                     this.offsetY = 0;
                 }
             },
-            /*
-            * Default formatter of label's content
-            */
+            /**
+             * Default formatter of label's content
+             * @private
+             */
             defaultFormatter: function () {
                 return 'Min: ' + this.min +
                     '<br>Max: ' + this.max +
                     '<br>Average: ' + this.average +
                     '<br>Bins: ' + this.bins;
             },
-            /*
-            * Set values for xAxisMin, xAxisMax, yAxisMin, yAxisMax, also
-            * when chart is inverted
-            */
+            /**
+             * Set values for xAxisMin, xAxisMax, yAxisMin, yAxisMax, also
+             * when chart is inverted
+             * @private
+             */
             getExtremes: function (xAxisMin, xAxisMax, yAxisMin, yAxisMax) {
                 return {
                     xAxisMin: Math.min(xAxisMax, xAxisMin),
@@ -539,9 +538,10 @@ H.extendAnnotation(Measure, null,
                     yAxisMax: Math.max(yAxisMax, yAxisMin)
                 };
             },
-            /*
-            * Definitions of calculations (min, max, average, bins)
-            */
+            /**
+             * Definitions of calculations (min, max, average, bins)
+             * @private
+             */
             min: function () {
                 var min = Infinity,
                     series = this.chart.series,

package/es-modules/annotations/types/Pitchfork.js

@@ -167,10 +167,11 @@ H.extendAnnotation(Pitchfork, InfinityLine,
     /**
      * A pitchfork annotation.
      *
-     * @extends annotations.infinityLine
      * @sample highcharts/annotations-advanced/pitchfork/
      *         Pitchfork
-     * @product highstock
+     *
+     * @extends      annotations.infinityLine
+     * @product      highstock
      * @optionparent annotations.pitchfork
      */
     {
@@ -178,7 +179,7 @@ H.extendAnnotation(Pitchfork, InfinityLine,
             /**
              * Inner background options.
              *
-             * @extends annotations.crookedLine.shapeOptions
+             * @extends   annotations.crookedLine.shapeOptions
              * @excluding height, r, type, width
              */
             innerBackground: {
@@ -188,7 +189,7 @@ H.extendAnnotation(Pitchfork, InfinityLine,
             /**
              * Outer background options.
              *
-             * @extends annotations.crookedLine.shapeOptions
+             * @extends   annotations.crookedLine.shapeOptions
              * @excluding height, r, type, width
              */
             outerBackground: {

package/es-modules/annotations/types/VerticalLine.js

@@ -9,7 +9,7 @@ var Annotation = H.Annotation,
  * @class
  * @extends Annotation
  * @memberOf Highcharts
- **/
+ */
 function VerticalLine() {
     H.Annotation.apply(this, arguments);
 }
@@ -106,11 +106,12 @@ H.extendAnnotation(VerticalLine, null,
     /**
      * A vertical line annotation.
      *
-     * @extends annotations.crookedLine
-     * @excluding labels, shapes, controlPointOptions
      * @sample highcharts/annotations-advanced/vertical-line/
      *         Vertical line
-     * @product highstock
+     *
+     * @extends      annotations.crookedLine
+     * @excluding    labels, shapes, controlPointOptions
+     * @product      highstock
      * @optionparent annotations.verticalLine
      */
     {
@@ -142,7 +143,7 @@ H.extendAnnotation(VerticalLine, null,
             /**
              * Connector options.
              *
-             * @extends annotations.crookedLine.shapeOptions
+             * @extends   annotations.crookedLine.shapeOptions
              * @excluding height, r, type, width
              */
             connector: {

package/es-modules/error-messages.js

@@ -99,6 +99,10 @@ H.errorMessages = {
         "title": "Invalid instrument",
         "text": "<h1>Invalid instrument</h1><p>This happens when you try to use a sonification instrument that is not valid. If you are using a predefined instrument, make sure your spelling is correct.</p>"
     },
+    "31": {
+        "title": "Non-unique point or node id",
+        "text": "<h1>Non-unique point or node id</h1><p>This error occurs when using the same <code>id</code> for two or more points or nodes.</p>"
+    },
     "meta": {
         "files": [
             "errors/10/readme.md",
@@ -122,7 +126,8 @@ H.errorMessages = {
             "errors/27/readme.md",
             "errors/28/readme.md",
             "errors/29/readme.md",
-            "errors/30/readme.md"
+            "errors/30/readme.md",
+            "errors/31/readme.md"
         ]
     }
 };

package/es-modules/indicators/indicators.src.js

@@ -250,10 +250,12 @@ seriesType(
                         );
 
                         for (i = 0; i < croppedData.xData.length; i++) {
+                            // (#10774)
                             croppedDataValues.push([
-                                croppedData.xData[i],
-                                croppedData.yData[i]
-                            ]);
+                                croppedData.xData[i]
+                            ].concat(
+                                H.splat(croppedData.yData[i])
+                            ));
                         }
 
                         oldFirstPointIndex = processedData.xData.indexOf(

package/es-modules/mixins/nodes.js

@@ -171,14 +171,22 @@ H.NodesMixin = {
                 [this.fromNode, this.toNode];
 
         others.forEach(function (linkOrNode) {
-            Point.prototype.setState.apply(linkOrNode, args);
-
-            if (!linkOrNode.isNode) {
-                if (linkOrNode.fromNode.graphic) {
-                    Point.prototype.setState.apply(linkOrNode.fromNode, args);
-                }
-                if (linkOrNode.toNode.graphic) {
-                    Point.prototype.setState.apply(linkOrNode.toNode, args);
+            if (linkOrNode.series) {
+                Point.prototype.setState.apply(linkOrNode, args);
+
+                if (!linkOrNode.isNode) {
+                    if (linkOrNode.fromNode.graphic) {
+                        Point.prototype.setState.apply(
+                            linkOrNode.fromNode,
+                            args
+                        );
+                    }
+                    if (linkOrNode.toNode.graphic) {
+                        Point.prototype.setState.apply(
+                            linkOrNode.toNode,
+                            args
+                        );
+                    }
                 }
             }
         });

package/es-modules/modules/accessibility/AccessibilityComponent.js

@@ -265,10 +265,10 @@ AccessibilityComponent.prototype = {
             var rectEl = el.getBoundingClientRect(),
                 rectDiv = div.getBoundingClientRect();
             return {
-                x: rectEl.x - rectDiv.x,
-                y: rectEl.y - rectDiv.y,
-                width: rectEl.width,
-                height: rectEl.height
+                x: rectEl.left - rectDiv.left,
+                y: rectEl.top - rectDiv.top,
+                width: rectEl.right - rectEl.left,
+                height: rectEl.bottom - rectEl.top
             };
         }
     },

package/es-modules/modules/accessibility/KeyboardNavigation.js

@@ -249,12 +249,21 @@ KeyboardNavigation.prototype = {
      */
     addExitAnchor: function () {
         var chart = this.chart,
+            exitAnchorWrapper = this.exitAnchorWrapper =
+                doc.createElement('div'),
             exitAnchor = this.exitAnchor = doc.createElement('h6'),
             keyboardNavigation = this,
             exitAnchorLabel = chart.langFormat(
                 'accessibility.svgContainerEnd', { chart: chart }
             );
 
+        exitAnchorWrapper.setAttribute('aria-hidden', 'false');
+        exitAnchorWrapper.setAttribute(
+            'class', 'highcharts-exit-anchor-wrapper'
+        );
+        exitAnchorWrapper.style.position = 'relative';
+        exitAnchorWrapper.style.outline = 'none';
+
         exitAnchor.setAttribute('tabindex', '0');
         exitAnchor.setAttribute('aria-label', exitAnchorLabel);
         exitAnchor.setAttribute('aria-hidden', false);
@@ -264,16 +273,18 @@ KeyboardNavigation.prototype = {
             position: 'absolute',
             width: '1px',
             height: '1px',
+            bottom: '5px', // Avoid scrollbars (#10637)
             zIndex: 0,
             overflow: 'hidden',
             outline: 'none'
         });
 
-        chart.renderTo.appendChild(exitAnchor);
+        exitAnchorWrapper.appendChild(exitAnchor);
+        chart.renderTo.appendChild(exitAnchorWrapper);
 
         // Update position on render
         this.unbindExitAnchorUpdate = addEvent(chart, 'render', function () {
-            this.renderTo.appendChild(exitAnchor);
+            this.renderTo.appendChild(exitAnchorWrapper);
         });
 
         // Handle focus
@@ -338,9 +349,11 @@ KeyboardNavigation.prototype = {
             this.unbindExitAnchorUpdate();
             delete this.unbindExitAnchorUpdate;
         }
-        if (this.exitAnchor && this.exitAnchor.parentNode) {
-            this.exitAnchor.parentNode.removeChild(this.exitAnchor);
+        if (this.exitAnchorWrapper && this.exitAnchorWrapper.parentNode) {
+            this.exitAnchorWrapper.parentNode
+                .removeChild(this.exitAnchorWrapper);
             delete this.exitAnchor;
+            delete this.exitAnchorWrapper;
         }
 
         // Remove keydown handler

package/es-modules/modules/accessibility/components/RangeSelectorComponent.js

@@ -184,7 +184,8 @@ H.extend(RangeSelectorComponent.prototype, /** @lends Highcharts.RangeSelectorCo
     /**
      * Get navigation for the range selector input boxes.
      * @private
-     * @return {Highcharts.KeyboardNavigationHandler} The module object.
+     * @return {Highcharts.KeyboardNavigationHandler}
+     *         The module object.
      */
     getRangeSelectorInputNavigation: function () {
         var chart = this.chart,
@@ -255,7 +256,7 @@ H.extend(RangeSelectorComponent.prototype, /** @lends Highcharts.RangeSelectorCo
     /**
      * Get keyboard navigation handlers for this component.
      * @return {Array<Highcharts.KeyboardNavigationHandler>}
-     *      List of module objects.
+     *         List of module objects.
      */
     getKeyboardNavigation: function () {
         return [

package/es-modules/modules/accessibility/components/SeriesComponent.js

@@ -30,6 +30,113 @@ H.Series.prototype.keyboardMoveVertical = true;
 });
 
 
+/**
+ * Keep track of forcing markers.
+ * @private
+ */
+H.addEvent(H.Series, 'render', function () {
+    var series = this,
+        chart = series.chart,
+        options = series.options,
+        a11yOptions = chart.options.accessibility || {},
+        points = series.points || [],
+        dataLength = points.length,
+        resetMarkerOptions = series.resetA11yMarkerOptions,
+        // We need markers for a11y
+        forceMarkers = a11yOptions.enabled &&
+            (
+                options.accessibility &&
+                options.accessibility.enabled
+            ) !== false &&
+            (
+                dataLength < a11yOptions.pointDescriptionThreshold ||
+                a11yOptions.pointDescriptionThreshold === false
+            );
+
+    if (forceMarkers) {
+        // If markers are explicitly disabled on series, replace with markers
+        // that have zero opacity.
+        if (options.marker && options.marker.enabled === false) {
+            series.a11yMarkersForced = true;
+            merge(true, series.options, {
+                marker: {
+                    enabled: true,
+                    states: {
+                        normal: {
+                            opacity: 0
+                        }
+                    }
+                }
+            });
+        }
+
+        // If we have point markers, we need to handle them
+        if (series._hasPointMarkers && series.points && series.points.length) {
+            var i = dataLength,
+                pointOptions;
+            while (i--) {
+                pointOptions = points[i].options;
+                if (pointOptions.marker) {
+                    if (pointOptions.marker.enabled) {
+                        // Make sure opacity is overridden to show enabled
+                        // markers
+                        merge(true, pointOptions.marker, {
+                            states: {
+                                normal: {
+                                    opacity: pointOptions.marker.states &&
+                                        pointOptions.marker.states.normal &&
+                                        pointOptions.marker.states.normal
+                                            .opacity || 1
+                                }
+                            }
+                        });
+                    } else {
+                        // Make sure hidden markers are enabled instead, and
+                        // opacity is out.
+                        merge(true, pointOptions.marker, {
+                            enabled: true,
+                            states: {
+                                normal: {
+                                    opacity: 0
+                                }
+                            }
+                        });
+                    }
+                }
+            }
+        }
+
+    } else if (series.a11yMarkersForced && resetMarkerOptions) {
+        // Series markers should not be forced, and we should reset to old
+        // options.
+        delete series.a11yMarkersForced;
+        merge(true, series.options, {
+            marker: {
+                enabled: resetMarkerOptions.enabled,
+                states: {
+                    normal: {
+                        opacity: resetMarkerOptions.states &&
+                            resetMarkerOptions.states.normal &&
+                            resetMarkerOptions.states.normal.opacity
+                    }
+                }
+            }
+        });
+    }
+});
+
+
+/**
+ * Keep track of options to reset markers to if no longer forced.
+ * @private
+ */
+H.addEvent(H.Series, 'afterSetOptions', function (e) {
+    this.resetA11yMarkerOptions = merge(
+        e.options.marker || {}, this.userOptions.marker || {}
+    );
+});
+
+
 /**
  * Get the index of a point in a series. This is needed when using e.g. data
  * grouping.
@@ -501,9 +608,10 @@ H.extend(SeriesComponent.prototype, /** @lends Highcharts.SeriesComponent */ {
 
 
     /**
-     * Called on first render/updates to the chart, including options changes.
+     * Called on chart render. It is necessary to do this for render in case
+     * markers change on zoom/pixel density.
      */
-    onChartUpdate: function () {
+    onChartRender: function () {
         var component = this,
             chart = this.chart;
         chart.series.forEach(function (series) {

package/es-modules/modules/accessibility/components/ZoomComponent.js

@@ -306,7 +306,7 @@ H.extend(ZoomComponent.prototype, /** @lends Highcharts.ZoomComponent */ {
     /**
      * Get keyboard navigation handlers for this component.
      * @return {Array<Highcharts.KeyboardNavigationHandler>}
-     *      List of module objects
+     *         List of module objects
      */
     getKeyboardNavigation: function () {
         return [

package/es-modules/modules/accessibility/options.js

@@ -63,7 +63,7 @@ var options = {
          * @type  {boolean|number}
          * @since 5.0.0
          */
-        pointDescriptionThreshold: 500, // set to false to disable
+        pointDescriptionThreshold: 200, // set to false to disable
 
         /**
          * Whether or not to add a shortcut button in the screen reader
@@ -372,6 +372,7 @@ var options = {
              * components can be added here.
              *
              * @since 7.1.0
+             * @type {Array<string>}
              */
             order: ['series', 'zoom', 'rangeSelector', 'chartMenu', 'legend'],
 

package/es-modules/modules/bellcurve.src.js

@@ -64,44 +64,42 @@ function normalDensity(x, mean, standardDeviation) {
  * @name Highcharts.seriesTypes.bellcurve
  *
  * @augments Highcharts.Series
- *
- * @mixes DerivedSeriesMixin
  */
 seriesType('bellcurve', 'areaspline'
 
     /**
- * A bell curve is an areaspline series which represents the probability density
- * function of the normal distribution. It calculates mean and standard
- * deviation of the base series data and plots the curve according to the
- * calculated parameters.
- *
- * @sample {highcharts} highcharts/demo/bellcurve/
- *         Bell curve
- *
- * @extends      plotOptions.areaspline
- * @since        6.0.0
- * @product      highcharts
- * @excluding    boostThreshold, connectNulls, stacking, pointInterval,
- *               pointIntervalUnit
- * @optionparent plotOptions.bellcurve
- */
+     * A bell curve is an areaspline series which represents the probability
+     * density function of the normal distribution. It calculates mean and
+     * standard deviation of the base series data and plots the curve according
+     * to the calculated parameters.
+     *
+     * @sample {highcharts} highcharts/demo/bellcurve/
+     *         Bell curve
+     *
+     * @extends      plotOptions.areaspline
+     * @since        6.0.0
+     * @product      highcharts
+     * @excluding    boostThreshold, connectNulls, stacking, pointInterval,
+     *               pointIntervalUnit
+     * @optionparent plotOptions.bellcurve
+     */
     , {
         /**
-    * This option allows to define the length of the bell curve. A unit of the
-    * length of the bell curve is standard deviation.
-    *
-    * @sample highcharts/plotoptions/bellcurve-intervals-pointsininterval
-    *         Intervals and points in interval
-    */
+         * This option allows to define the length of the bell curve. A unit of
+         * the length of the bell curve is standard deviation.
+         *
+         * @sample highcharts/plotoptions/bellcurve-intervals-pointsininterval
+         *         Intervals and points in interval
+         */
         intervals: 3,
 
         /**
-    * Defines how many points should be plotted within 1 interval. See
-    * `plotOptions.bellcurve.intervals`.
-    *
-    * @sample highcharts/plotoptions/bellcurve-intervals-pointsininterval
-    *         Intervals and points in interval
-    */
+         * Defines how many points should be plotted within 1 interval. See
+         * `plotOptions.bellcurve.intervals`.
+         *
+         * @sample highcharts/plotoptions/bellcurve-intervals-pointsininterval
+         *         Intervals and points in interval
+         */
         pointsInInterval: 3,
 
         marker: {

package/es-modules/modules/boost/boost-init.js

@@ -136,6 +136,12 @@ function init() {
             // If we're rendering per. series we should create the marker groups
             // as usual.
             if (!chart.isChartSeriesBoosting()) {
+                // If all series were boosting, but are not anymore
+                // restore private markerGroup
+                if (this.markerGroup === chart.markerGroup) {
+                    this.markerGroup = undefined;
+                }
+
                 this.markerGroup = series.plotGroup(
                     'markerGroup',
                     'markers',
@@ -144,6 +150,14 @@ function init() {
                     chart.seriesGroup
                 );
             } else {
+                // If series has a private markeGroup, remove that
+                // and use common markerGroup
+                if (
+                    this.markerGroup &&
+                    this.markerGroup !== chart.markerGroup
+                ) {
+                    this.markerGroup.destroy();
+                }
                 // Use a single group for the markers
                 this.markerGroup = chart.markerGroup;
 

package/es-modules/modules/boost/boost-overrides.js

@@ -408,11 +408,21 @@ Series.prototype.hasExtremes = function (checkX) {
     var options = this.options,
         data = options.data,
         xAxis = this.xAxis && this.xAxis.options,
-        yAxis = this.yAxis && this.yAxis.options;
+        yAxis = this.yAxis && this.yAxis.options,
+        colorAxis = this.colorAxis && this.colorAxis.options;
 
     return data.length > (options.boostThreshold || Number.MAX_VALUE) &&
-            isNumber(yAxis.min) && isNumber(yAxis.max) &&
-            (!checkX || (isNumber(xAxis.min) && isNumber(xAxis.max)));
+            // Defined yAxis extremes
+            isNumber(yAxis.min) &&
+            isNumber(yAxis.max) &&
+            // Defined (and required) xAxis extremes
+            (!checkX ||
+                (isNumber(xAxis.min) && isNumber(xAxis.max))
+            ) &&
+            // Defined (e.g. heatmap) colorAxis extremes
+            (!colorAxis ||
+                (isNumber(colorAxis.min) && isNumber(colorAxis.max))
+            );
 };
 
 /**