highcharts 7.1.1 → 7.1.2

package/es-modules/parts/Dynamics.js

@@ -44,14 +44,18 @@ H.cleanRecursively = function (newer, older) {
     objectEach(newer, function (val, key) {
         var ob;
 
-        // Dive into objects
-        if (isObject(newer[key], true) && older[key]) {
+        // Dive into objects (except DOM nodes)
+        if (
+            isObject(newer[key], true) &&
+            !newer.nodeType && // #10044
+            older[key]
+        ) {
             ob = H.cleanRecursively(newer[key], older[key]);
             if (Object.keys(ob).length) {
                 result[key] = ob;
             }
 
-        // Arrays or primitives are copied directly
+        // Arrays, primitives and DOM nodes are copied directly
         } else if (isObject(newer[key]) || newer[key] !== older[key]) {
             result[key] = newer[key];
         }
@@ -382,16 +386,16 @@ extend(Chart.prototype, /** @lends Highcharts.Chart.prototype */ {
      *        Whether to redraw the chart.
      *
      * @param {boolean} [oneToOne=false]
-     *        When `true`, the `series`, `xAxis` and `yAxis` collections will
-     *        be updated one to one, and items will be either added or removed
-     *        to match the new updated options. For example, if the chart has
-     *        two series and we call `chart.update` with a configuration
-     *        containing three series, one will be added. If we call
-     *        `chart.update` with one series, one will be removed. Setting an
-     *        empty `series` array will remove all series, but leaving out the
-     *        `series` property will leave all series untouched. If the series
-     *        have id's, the new series options will be matched by id, and the
-     *        remaining ones removed.
+     *        When `true`, the `series`, `xAxis`, `yAxis` and `annotations`
+     *        collections will be updated one to one, and items will be either
+     *        added or removed to match the new updated options. For example,
+     *        if the chart has two series and we call `chart.update` with a
+     *        configuration containing three series, one will be added. If we
+     *        call `chart.update` with one series, one will be removed. Setting
+     *        an empty `series` array will remove all series, but leaving out
+     *        the`series` property will leave all series untouched. If the
+     *        series have id's, the new series options will be matched by id,
+     *        and the remaining ones removed.
      *
      * @param {boolean|Highcharts.AnimationOptionsObject} [animation=true]
      *        Whether to apply animation, and optionally animation
@@ -426,6 +430,8 @@ extend(Chart.prototype, /** @lends Highcharts.Chart.prototype */ {
 
         options = H.cleanRecursively(options, chart.options);
 
+        merge(true, chart.userOptions, options);
+
         // If the top-level chart option is present, some special updates are
         // required
         optionsChart = options.chart;
@@ -489,6 +495,13 @@ extend(Chart.prototype, /** @lends Highcharts.Chart.prototype */ {
             merge(true, this.options.plotOptions, options.plotOptions);
         }
 
+        // Maintaining legacy global time. If the chart is instanciated first
+        // with global time, then updated with time options, we need to create a
+        // new Time instance to avoid mutating the global time (#10536).
+        if (options.time && this.time === H.time) {
+            this.time = new H.Time(options.time);
+        }
+
         // Some option stuctures correspond one-to-one to chart objects that
         // have update methods, for example
         // options.credits => chart.credits
@@ -555,14 +568,19 @@ extend(Chart.prototype, /** @lends Highcharts.Chart.prototype */ {
                     }
 
                     // If oneToOne and no matching item is found, add one
-                    if (!item && oneToOne) {
-                        if (coll === 'series') {
-                            chart.addSeries(newOptions, false)
-                                .touched = true;
-                        } else if (coll === 'xAxis' || coll === 'yAxis') {
-                            chart.addAxis(newOptions, coll === 'xAxis', false)
-                                .touched = true;
-                        }
+                    if (!item && oneToOne && chart.collectionsWithInit[coll]) {
+                        chart.collectionsWithInit[coll][0].apply(
+                            chart,
+                            // [newOptions, ...extraArguments, redraw=false]
+                            [
+                                newOptions
+                            ].concat(
+                                // Not all initializers require extra args
+                                chart.collectionsWithInit[coll][1] || []
+                            ).concat([
+                                false
+                            ])
+                        ).touched = true;
                     }
 
                 });
@@ -657,6 +675,21 @@ extend(Chart.prototype, /** @lends Highcharts.Chart.prototype */ {
 
 });
 
+/**
+ * These collections (arrays) implement `Chart.addSomethig` method used in
+ * chart.update() to create new object in the collection. Equivalent for
+ * deleting is resolved by simple `Somethig.remove()`.
+ *
+ * Note: We need to define these references after initializers are bound to
+ * chart's prototype.
+ */
+Chart.prototype.collectionsWithInit = {
+    // collectionName: [ initializingMethod, [extraArguments] ]
+    xAxis: [Chart.prototype.addAxis, [true]],
+    yAxis: [Chart.prototype.addAxis, [false]],
+    series: [Chart.prototype.addSeries]
+};
+
 // extend the Point prototype for dynamic methods
 extend(Point.prototype, /** @lends Highcharts.Point.prototype */ {
     /**
@@ -856,8 +889,8 @@ extend(Series.prototype, /** @lends Series.prototype */ {
             names = xAxis && xAxis.hasNames && xAxis.names,
             dataOptions = seriesOptions.data,
             point,
-            isInTheMiddle,
             xData = series.xData,
+            isInTheMiddle,
             i,
             x;
 
@@ -1109,7 +1142,8 @@ extend(Series.prototype, /** @lends Series.prototype */ {
             groups = [
                 'group',
                 'markerGroup',
-                'dataLabelsGroup'
+                'dataLabelsGroup',
+                'transformGroup'
             ],
             preserve = [
                 'navigatorSeries',
@@ -1130,7 +1164,18 @@ extend(Series.prototype, /** @lends Series.prototype */ {
                 'points',
                 'processedXData',
                 'processedYData',
-                'xIncrement'
+                'xIncrement',
+                '_hasPointMarkers',
+                '_hasPointLabels',
+
+                // Map specific, consider moving it to series-specific preserve-
+                // properties (#10617)
+                'mapMap',
+                'mapData',
+                'minY',
+                'maxY',
+                'minX',
+                'maxX'
             );
             if (options.visible !== false) {
                 preserve.push('area', 'graph');
@@ -1197,13 +1242,15 @@ extend(Series.prototype, /** @lends Series.prototype */ {
             } else {
                 if (
                     seriesOptions.marker &&
-                    seriesOptions.marker.enabled === false
+                    seriesOptions.marker.enabled === false &&
+                    !series._hasPointMarkers
                 ) {
                     kinds.graphic = 1;
                 }
                 if (
                     seriesOptions.dataLabels &&
-                    seriesOptions.dataLabels.enabled === false
+                    seriesOptions.dataLabels.enabled === false &&
+                    !series._hasPointLabels
                 ) {
                     kinds.dataLabel = 1;
                 }

package/es-modules/parts/FlagsSeries.js

@@ -353,7 +353,8 @@ seriesType(
                 outsideRight,
                 yAxis = series.yAxis,
                 boxesMap = {},
-                boxes = [];
+                boxes = [],
+                centered;
 
             i = points.length;
             while (i--) {
@@ -375,6 +376,7 @@ seriesType(
                 // skip connectors for higher level stacked points
                 point.anchorX = stackIndex ? undefined : point.plotX;
                 anchorY = stackIndex ? undefined : point.plotY;
+                centered = shape !== 'flag';
 
                 graphic = point.graphic;
 
@@ -401,7 +403,7 @@ seriesType(
                         }
 
                         graphic.attr({
-                            align: shape === 'flag' ? 'left' : 'center',
+                            align: centered ? 'center' : 'left',
                             width: options.width,
                             height: options.height,
                             'text-align': options.textAlign
@@ -442,7 +444,7 @@ seriesType(
                     if (!options.allowOverlapX) {
                         if (!boxesMap[point.plotX]) {
                             boxesMap[point.plotX] = {
-                                align: 0,
+                                align: centered ? 0.5 : 0,
                                 size: graphic.width,
                                 target: plotX,
                                 anchorX: plotX
@@ -483,7 +485,7 @@ seriesType(
                         point.graphic[
                             point.graphic.isNew ? 'attr' : 'animate'
                         ]({
-                            x: box.pos,
+                            x: box.pos + box.align * box.size,
                             anchorX: point.anchorX
                         });
                         // Hide flag when its box position is not specified

package/es-modules/parts/Globals.js

@@ -1,9 +1,14 @@
 /* *
- * (c) 2010-2019 Torstein Honsi
  *
- * License: www.highcharts.com/license
- */
-
+ *  (c) 2010-2019 Torstein Honsi
+ *
+ *  License: www.highcharts.com/license
+ *
+ *  !!!!!!! SOURCE GETS TRANSPILED BY TYPESCRIPT. EDIT TS FILE ONLY. !!!!!!!
+ *
+ * */
+'use strict';
+/* globals Image, window */
 /**
  * Reference to the global SVGElement class as a workaround for a name conflict
  * in the Highcharts namespace.
@@ -13,38 +18,21 @@
  *
  * @see https://developer.mozilla.org/en-US/docs/Web/API/SVGElement
  */
-
-'use strict';
-
-/* global win, window */
-
 // glob is a temporary fix to allow our es-modules to work.
 var glob = typeof win === 'undefined' ?
-        (typeof window !== 'undefined' ? window : {}) :
-        win,
-    doc = glob.document,
-    SVG_NS = 'http://www.w3.org/2000/svg',
-    userAgent = (glob.navigator && glob.navigator.userAgent) || '',
-    svg = (
-        doc &&
-        doc.createElementNS &&
-        !!doc.createElementNS(SVG_NS, 'svg').createSVGRect
-    ),
-    isMS = /(edge|msie|trident)/i.test(userAgent) && !glob.opera,
-    isFirefox = userAgent.indexOf('Firefox') !== -1,
-    isChrome = userAgent.indexOf('Chrome') !== -1,
-    hasBidiBug = (
-        isFirefox &&
-        parseInt(userAgent.split('Firefox/')[1], 10) < 4 // issue #38
-    );
-
-var Highcharts = {
+    (typeof window !== 'undefined' ? window : {}) :
+    win, doc = glob.document, SVG_NS = 'http://www.w3.org/2000/svg', userAgent = (glob.navigator && glob.navigator.userAgent) || '', svg = (doc &&
+    doc.createElementNS &&
+    !!doc.createElementNS(SVG_NS, 'svg').createSVGRect), isMS = /(edge|msie|trident)/i.test(userAgent) && !glob.opera, isFirefox = userAgent.indexOf('Firefox') !== -1, isChrome = userAgent.indexOf('Chrome') !== -1, hasBidiBug = (isFirefox &&
+    parseInt(userAgent.split('Firefox/')[1], 10) < 4 // issue #38
+);
+var H = {
     product: '@product.name@',
     version: '@product.version@',
     deg2rad: Math.PI * 2 / 360,
     doc: doc,
     hasBidiBug: hasBidiBug,
-    hasTouch: doc && doc.documentElement.ontouchstart !== undefined,
+    hasTouch: doc && typeof doc.documentElement.ontouchstart !== 'undefined',
     isMS: isMS,
     isWebKit: userAgent.indexOf('AppleWebKit') !== -1,
     isFirefox: isFirefox,
@@ -58,19 +46,16 @@ var Highcharts = {
     svg: svg,
     win: glob,
     marginNames: ['plotTop', 'marginRight', 'marginBottom', 'plotLeft'],
-    noop: function () {
-        return undefined;
-    },
+    noop: function () { },
     /**
      * An array containing the current chart objects in the page. A chart's
      * position in the array is preserved throughout the page's lifetime. When
      * a chart is destroyed, the array item becomes `undefined`.
      *
      * @name Highcharts.charts
-     * @type {Array<Highcharts.Chart>}
+     * @type {Array<Highcharts.Chart|undefined>}
      */
     charts: [],
-
     /**
      * A hook for defining additional date format specifiers. New
      * specifiers are defined as key-value pairs by using the
@@ -86,5 +71,4 @@ var Highcharts = {
      */
     dateFormats: {}
 };
-
-export default Highcharts;
+export default H;