highcharts 7.1.1 → 7.1.2

package/es-modules/annotations/controllable/ControllableCircle.js

@@ -36,6 +36,8 @@ H.merge(
          */
         type: 'circle',
 
+        translate: controllableMixin.translateShape,
+
         render: function (parent) {
             var attrs = this.attrsFromOptions(this.options);
 
@@ -68,16 +70,6 @@ H.merge(
             controllableMixin.redraw.call(this, animation);
         },
 
-        translate: function (dx, dy) {
-            var annotationOptions = this.annotation.userOptions,
-                shapeOptions = annotationOptions[this.collection][this.index];
-
-            this.translatePoint(dx, dy, 0);
-
-            // Options stored in chart:
-            shapeOptions.point = this.options.point;
-        },
-
         /**
          * Set the radius.
          *

package/es-modules/annotations/controllable/ControllableImage.js

@@ -47,6 +47,8 @@ H.merge(
          */
         type: 'image',
 
+        translate: controllableMixin.translateShape,
+
         render: function (parent) {
             var attrs = this.attrsFromOptions(this.options),
                 options = this.options;
@@ -84,16 +86,6 @@ H.merge(
             this.graphic.placed = Boolean(position);
 
             controllableMixin.redraw.call(this, animation);
-        },
-
-        translate: function (dx, dy) {
-            var annotationOptions = this.annotation.userOptions,
-                shapeOptions = annotationOptions[this.collection][this.index];
-
-            this.translatePoint(dx, dy, 0);
-
-            // Options stored in chart:
-            shapeOptions.point = this.options.point;
         }
     }
 );

package/es-modules/annotations/controllable/ControllableLabel.js

@@ -5,15 +5,50 @@ import './../../parts/SvgRenderer.js';
 import controllableMixin from './controllableMixin.js';
 import MockPoint from './../MockPoint.js';
 
+
+/**
+ * @private
+ * @interface Highcharts.AnnotationAnchorObject
+ *//**
+ * Relative to the plot area position
+ * @name Highcharts.AnnotationAnchorObject#relativePosition
+ * @type {Highcharts.AnnotationAnchorPositionObject}
+ *//**
+ * Absolute position
+ * @name Highcharts.AnnotationAnchorObject#absolutePosition
+ * @type {Highcharts.AnnotationAnchorPositionObject}
+ */
+
+/**
+ * An object which denotes an anchor position
+ *
+ * @private
+ * @interface Highcharts.AnnotationAnchorPositionObject
+ *//**
+ * @name Highcharts.AnnotationAnchorPositionObject#x
+ * @property {number}
+ *//**
+ * @name Highcharts.AnnotationAnchorPositionObject#y
+ * @property {number}
+ *//**
+ * @name Highcharts.AnnotationAnchorPositionObject#height
+ * @property {number}
+ *//**
+ * @name Highcharts.AnnotationAnchorPositionObject#width
+ * @property {number}
+ */
+
 /**
  * A controllable label class.
  *
+ * @private
  * @class
+ * @name Annotation.ControllableLabel
+ *
  * @mixes Annotation.controllableMixin
- * @memberOf Annotation
  *
  * @param {Highcharts.Annotation} annotation an annotation instance
- * @param {Object} options a label's options
+ * @param {object} options a label's options
  * @param {number} index of the label
  **/
 function ControllableLabel(annotation, options, index) {
@@ -25,7 +60,7 @@ function ControllableLabel(annotation, options, index) {
  * Shapes which do not have background - the object is used for proper
  * setting of the contrast color.
  *
- * @type {Array<String>}
+ * @type {Array<string>}
  */
 ControllableLabel.shapesWithoutBackground = ['connector'];
 
@@ -195,16 +230,24 @@ H.merge(
          * @param {number} dy translation for y coordinate
          **/
         translate: function (dx, dy) {
-            var annotationOptions = this.annotation.userOptions,
-                labelOptions = annotationOptions[this.collection][this.index];
+            var chart = this.annotation.chart,
+                // Annotation.options
+                labelOptions = this.annotation.userOptions,
+                // Chart.options.annotations
+                annotationIndex = chart.annotations.indexOf(this.annotation),
+                chartAnnotations = chart.options.annotations,
+                chartOptions = chartAnnotations[annotationIndex];
 
             // Local options:
             this.options.x += dx;
             this.options.y += dy;
 
             // Options stored in chart:
-            labelOptions.x = this.options.x;
-            labelOptions.y = this.options.y;
+            chartOptions[this.collection][this.index].x = this.options.x;
+            chartOptions[this.collection][this.index].y = this.options.y;
+
+            labelOptions[this.collection][this.index].x = this.options.x;
+            labelOptions[this.collection][this.index].y = this.options.y;
         },
 
         render: function (parent) {
@@ -312,8 +355,9 @@ H.merge(
         /**
          * Returns the label position relative to its anchor.
          *
-         * @param {Annotation.controllableMixin.Anchor} anchor
-         * @return {Annotation.controllableMixin.Position|null} position
+         * @param {Highcharts.AnnotationAnchorObject} anchor
+         *
+         * @return {Highcharts.AnnotationAnchorPositionObject|null} position
          */
         position: function (anchor) {
             var item = this.graphic,
@@ -407,6 +451,7 @@ H.merge(
 
 /**
  * General symbol definition for labels with connector
+ * @private
  */
 H.SVGRenderer.prototype.symbols.connector = function (x, y, w, h, options) {
     var anchorX = options && options.anchorX,

package/es-modules/annotations/controllable/ControllableRect.js

@@ -45,6 +45,8 @@ H.merge(
          */
         type: 'rect',
 
+        translate: controllableMixin.translateShape,
+
         render: function (parent) {
             var attrs = this.attrsFromOptions(this.options);
 
@@ -76,16 +78,6 @@ H.merge(
             this.graphic.placed = Boolean(position);
 
             controllableMixin.redraw.call(this, animation);
-        },
-
-        translate: function (dx, dy) {
-            var annotationOptions = this.annotation.userOptions,
-                shapeOptions = annotationOptions[this.collection][this.index];
-
-            this.translatePoint(dx, dy, 0);
-
-            // Options stored in chart:
-            shapeOptions.point = this.options.point;
         }
     }
 );

package/es-modules/annotations/controllable/controllableMixin.js

@@ -9,6 +9,7 @@ import MockPoint from './../MockPoint.js';
  * It provides methods for handling points, control points
  * and points transformations.
  *
+ * @private
  * @mixin
  * @memberOf Annotation
  */
@@ -142,10 +143,13 @@ var controllableMixin = {
     /**
      * Map point's options to a point-like object.
      *
-     * @param {Annotation.MockPoint.Options} pointOptions point's options
-     * @param {Annotation.PointLike} point a point like instance
-     * @return {Annotation.PointLike|null} if the point is
-     *         found/set returns this point, otherwise null
+     * @param {Highcharts.MockPointOptionsObject} pointOptions
+     *        point's options
+     * @param {Highcharts.PointLike} point
+     *        a point like instance
+     *
+     * @return {Highcharts.PointLike|null}
+     *         if the point is found/set returns this point, otherwise null
      */
     point: function (pointOptions, point) {
         if (pointOptions && pointOptions.series) {
@@ -331,6 +335,30 @@ var controllableMixin = {
         this.transformPoint('translate', null, null, dx, dy, i);
     },
 
+    /**
+     * Translate shape within controllable item.
+     * Replaces `controllable.translate` method.
+     *
+     * @param {number} dx translation for x coordinate
+     * @param {number} dy translation for y coordinate
+     */
+    translateShape: function (dx, dy) {
+        var chart = this.annotation.chart,
+            // Annotation.options
+            shapeOptions = this.annotation.userOptions,
+            // Chart.options.annotations
+            annotationIndex = chart.annotations.indexOf(this.annotation),
+            chartOptions = chart.options.annotations[annotationIndex];
+
+        this.translatePoint(dx, dy, 0);
+
+        // Options stored in:
+        // - chart (for exporting)
+        // - current config (for redraws)
+        chartOptions[this.collection][this.index].point = this.options.point;
+        shapeOptions[this.collection][this.index].point = this.options.point;
+    },
+
     /**
      * Rotate a controllable.
      *

package/es-modules/annotations/controllable/markerMixin.js

@@ -100,6 +100,7 @@ var createMarkerSetter = function (markerType) {
 };
 
 /**
+ * @private
  * @mixin
  */
 var markerMixin = {

package/es-modules/annotations/eventEmitterMixin.js

@@ -10,6 +10,7 @@ var fireEvent = H.fireEvent;
  *   The units of the distance are specific to a transformation,
  *   e.g. for rotation they are radians, for scaling they are scale factors.
  *
+ * @private
  * @mixin
  * @memberOf Annotation
  */

package/es-modules/annotations/navigationBindings.js

@@ -1,8 +1,10 @@
-/**
- * (c) 2009-2017 Highsoft, Black Label
+/* *
  *
- * License: www.highcharts.com/license
- */
+ *  (c) 2009-2017 Highsoft, Black Label
+ *
+ *  License: www.highcharts.com/license
+ *
+ * */
 'use strict';
 import H from '../parts/Globals.js';
 import chartNavigationMixin from '../mixins/navigation.js';
@@ -218,13 +220,15 @@ extend(H.NavigationBindings.prototype, {
         });
 
         objectEach(options.events || {}, function (callback, eventName) {
-            navigation.eventsToUnbind.push(
-                addEvent(
-                    navigation,
-                    eventName,
-                    callback
-                )
-            );
+            if (H.isFunction(callback)) {
+                navigation.eventsToUnbind.push(
+                    addEvent(
+                        navigation,
+                        eventName,
+                        callback
+                    )
+                );
+            }
         });
 
         navigation.eventsToUnbind.push(