highcharts 9.3.1 → 10.0.0

package/es-modules/Accessibility/KeyboardNavigation.js

@@ -33,7 +33,7 @@ var getElement = HTMLUtilities.getElement;
  * @class
  * @param {Highcharts.Chart} chart
  *        Chart object
- * @param {object} components
+ * @param {Object} components
  *        Map of component names to AccessibilityComponent objects.
  * @name Highcharts.KeyboardNavigation
  */
@@ -69,7 +69,7 @@ var KeyboardNavigation = /** @class */ (function () {
      * @private
      * @param {Highcharts.Chart} chart
      *        Chart object
-     * @param {object} components
+     * @param {Object} components
      *        Map of component names to AccessibilityComponent objects.
      */
     KeyboardNavigation.prototype.init = function (chart, components) {
@@ -111,7 +111,8 @@ var KeyboardNavigation = /** @class */ (function () {
             order.length) {
             // We (still) have keyboard navigation. Update module list
             this.modules = order.reduce(function (modules, componentName) {
-                var navModules = components[componentName].getKeyboardNavigation();
+                var navModules = components[componentName]
+                    .getKeyboardNavigation();
                 return modules.concat(navModules);
             }, []);
             this.updateExitAnchor();
@@ -122,6 +123,64 @@ var KeyboardNavigation = /** @class */ (function () {
             this.removeExitAnchor();
         }
     };
+    /**
+     * We use an exit anchor to move focus out of chart whenever we want, by
+     * setting focus to this div and not preventing the default tab action. We
+     * also use this when users come back into the chart by tabbing back, in
+     * order to navigate from the end of the chart.
+     * @private
+     */
+    KeyboardNavigation.prototype.updateExitAnchor = function () {
+        var endMarkerId = "highcharts-end-of-chart-marker-" + this.chart.index, endMarker = getElement(endMarkerId);
+        this.removeExitAnchor();
+        if (endMarker) {
+            this.makeElementAnExitAnchor(endMarker);
+            this.exitAnchor = endMarker;
+        }
+        else {
+            this.createExitAnchor();
+        }
+    };
+    /**
+     * Move to prev/next module.
+     * @private
+     * @param {number} direction
+     * Direction to move. +1 for next, -1 for prev.
+     * @return {boolean}
+     * True if there was a valid module in direction.
+     */
+    KeyboardNavigation.prototype.move = function (direction) {
+        var curModule = this.modules && this.modules[this.currentModuleIx];
+        if (curModule && curModule.terminate) {
+            curModule.terminate(direction);
+        }
+        // Remove existing focus border if any
+        if (this.chart.focusElement) {
+            this.chart.focusElement.removeFocusBorder();
+        }
+        this.currentModuleIx += direction;
+        var newModule = this.modules && this.modules[this.currentModuleIx];
+        if (newModule) {
+            if (newModule.validate && !newModule.validate()) {
+                return this.move(direction); // Invalid module, recurse
+            }
+            if (newModule.init) {
+                newModule.init(direction); // Valid module, init it
+                return true;
+            }
+        }
+        // No module
+        this.currentModuleIx = 0; // Reset counter
+        // Set focus to chart or exit anchor depending on direction
+        this.exiting = true;
+        if (direction > 0) {
+            this.exitAnchor.focus();
+        }
+        else {
+            this.tabindexContainer.focus();
+        }
+        return false;
+    };
     /**
      * Function to run on container focus
      * @private
@@ -135,29 +194,37 @@ var KeyboardNavigation = /** @class */ (function () {
         if (!this.exiting &&
             !this.tabbingInBackwards &&
             !this.isClickingChart &&
-            !focusComesFromChart &&
-            this.modules[0]) {
-            this.modules[0].init(1);
+            !focusComesFromChart) {
+            var ix = this.getFirstValidModuleIx();
+            if (ix !== null) {
+                this.currentModuleIx = ix;
+                this.modules[ix].init(1);
+            }
         }
         this.exiting = false;
     };
     /**
-     * Reset chart navigation state if we click outside the chart and it's
-     * not already reset.
+     * Reset chart navigation state if we mouse click and it's not already
+     * reset. Reset fully if outside the chart, otherwise just hide focus
+     * indicator.
      * @private
      */
     KeyboardNavigation.prototype.onMouseUp = function () {
         delete this.isClickingChart;
-        if (!this.keyboardReset && !this.pointerIsOverChart) {
-            var chart = this.chart, curMod = this.modules &&
-                this.modules[this.currentModuleIx || 0];
-            if (curMod && curMod.terminate) {
-                curMod.terminate();
+        if (!this.keyboardReset) {
+            var chart = this.chart;
+            if (!this.pointerIsOverChart) {
+                var curMod = this.modules &&
+                    this.modules[this.currentModuleIx || 0];
+                if (curMod && curMod.terminate) {
+                    curMod.terminate();
+                }
+                this.currentModuleIx = 0;
             }
             if (chart.focusElement) {
                 chart.focusElement.removeFocusBorder();
+                delete chart.focusElement;
             }
-            this.currentModuleIx = 0;
             this.keyboardReset = true;
         }
     };
@@ -183,10 +250,10 @@ var KeyboardNavigation = /** @class */ (function () {
                 preventDefault = true;
             }
             else if (response === curNavModule.response.prev) {
-                preventDefault = this.prev();
+                preventDefault = this.move(-1);
             }
             else if (response === curNavModule.response.next) {
-                preventDefault = this.next();
+                preventDefault = this.move(1);
             }
             if (preventDefault) {
                 e.preventDefault();
@@ -194,78 +261,6 @@ var KeyboardNavigation = /** @class */ (function () {
             }
         }
     };
-    /**
-     * Go to previous module.
-     * @private
-     */
-    KeyboardNavigation.prototype.prev = function () {
-        return this.move(-1);
-    };
-    /**
-     * Go to next module.
-     * @private
-     */
-    KeyboardNavigation.prototype.next = function () {
-        return this.move(1);
-    };
-    /**
-     * Move to prev/next module.
-     * @private
-     * @param {number} direction
-     * Direction to move. +1 for next, -1 for prev.
-     * @return {boolean}
-     * True if there was a valid module in direction.
-     */
-    KeyboardNavigation.prototype.move = function (direction) {
-        var curModule = this.modules && this.modules[this.currentModuleIx];
-        if (curModule && curModule.terminate) {
-            curModule.terminate(direction);
-        }
-        // Remove existing focus border if any
-        if (this.chart.focusElement) {
-            this.chart.focusElement.removeFocusBorder();
-        }
-        this.currentModuleIx += direction;
-        var newModule = this.modules && this.modules[this.currentModuleIx];
-        if (newModule) {
-            if (newModule.validate && !newModule.validate()) {
-                return this.move(direction); // Invalid module, recurse
-            }
-            if (newModule.init) {
-                newModule.init(direction); // Valid module, init it
-                return true;
-            }
-        }
-        // No module
-        this.currentModuleIx = 0; // Reset counter
-        // Set focus to chart or exit anchor depending on direction
-        this.exiting = true;
-        if (direction > 0) {
-            this.exitAnchor.focus();
-        }
-        else {
-            this.tabindexContainer.focus();
-        }
-        return false;
-    };
-    /**
-     * We use an exit anchor to move focus out of chart whenever we want, by
-     * setting focus to this div and not preventing the default tab action. We
-     * also use this when users come back into the chart by tabbing back, in
-     * order to navigate from the end of the chart.
-     * @private
-     */
-    KeyboardNavigation.prototype.updateExitAnchor = function () {
-        var endMarkerId = 'highcharts-end-of-chart-marker-' + this.chart.index, endMarker = getElement(endMarkerId);
-        this.removeExitAnchor();
-        if (endMarker) {
-            this.makeElementAnExitAnchor(endMarker);
-            this.exitAnchor = endMarker;
-        }
-        else {
-            this.createExitAnchor();
-        }
-    };
     /**
      * Chart container should have tabindex if navigation is enabled.
      * @private
@@ -290,6 +285,17 @@ var KeyboardNavigation = /** @class */ (function () {
         }
     };
     /**
+     * Add new exit anchor to the chart.
+     * @private
+     */
+    KeyboardNavigation.prototype.createExitAnchor = function () {
+        var chart = this.chart, exitAnchor = this.exitAnchor = doc.createElement('div');
+        chart.renderTo.appendChild(exitAnchor);
+        this.makeElementAnExitAnchor(exitAnchor);
+    };
+    /**
+     * Add attributes and events to an element to make it function as an
+     * exit anchor.
      * @private
      */
     KeyboardNavigation.prototype.makeElementAnExitAnchor = function (el) {
@@ -301,26 +307,17 @@ var KeyboardNavigation = /** @class */ (function () {
         this.addExitAnchorEventsToEl(el);
     };
     /**
-     * Add new exit anchor to the chart.
-     *
-     * @private
-     */
-    KeyboardNavigation.prototype.createExitAnchor = function () {
-        var chart = this.chart, exitAnchor = this.exitAnchor = doc.createElement('div');
-        chart.renderTo.appendChild(exitAnchor);
-        this.makeElementAnExitAnchor(exitAnchor);
-    };
-    /**
+     * Destroy the exit anchor and remove from DOM.
      * @private
      */
     KeyboardNavigation.prototype.removeExitAnchor = function () {
         if (this.exitAnchor && this.exitAnchor.parentNode) {
-            this.exitAnchor.parentNode
-                .removeChild(this.exitAnchor);
+            this.exitAnchor.parentNode.removeChild(this.exitAnchor);
             delete this.exitAnchor;
         }
     };
     /**
+     * Add focus handler to exit anchor element.
      * @private
      */
     KeyboardNavigation.prototype.addExitAnchorEventsToEl = function (element) {
@@ -328,6 +325,9 @@ var KeyboardNavigation = /** @class */ (function () {
         this.eventProvider.addEvent(element, 'focus', function (ev) {
             var e = ev || win.event, focusComesFromChart = (e.relatedTarget &&
                 chart.container.contains(e.relatedTarget)), comingInBackwards = !(focusComesFromChart || keyboardNavigation.exiting);
+            if (chart.focusElement) {
+                delete chart.focusElement;
+            }
             if (comingInBackwards) {
                 // Focus the container instead
                 keyboardNavigation.tabbingInBackwards = true;
@@ -345,7 +345,7 @@ var KeyboardNavigation = /** @class */ (function () {
                     if (curModule &&
                         curModule.validate && !curModule.validate()) {
                         // Invalid. Try moving backwards to find next valid.
-                        keyboardNavigation.prev();
+                        keyboardNavigation.move(-1);
                     }
                     else if (curModule) {
                         // We have a valid module, init it
@@ -359,6 +359,21 @@ var KeyboardNavigation = /** @class */ (function () {
             }
         });
     };
+    /**
+     * Get the ix of the first module that either does not require validation or
+     * validates positively.
+     * @private
+     */
+    KeyboardNavigation.prototype.getFirstValidModuleIx = function () {
+        var len = this.modules.length;
+        for (var i = 0; i < len; ++i) {
+            var mod = this.modules[i];
+            if (!mod.validate || mod.validate()) {
+                return i;
+            }
+        }
+        return null;
+    };
     /**
      * Remove all traces of keyboard navigation.
      * @private
@@ -394,6 +409,7 @@ var KeyboardNavigation = /** @class */ (function () {
      * */
     /* eslint-disable valid-jsdoc */
     /**
+     * Composition function.
      * @private
      */
     function compose(ChartClass) {

package/es-modules/Accessibility/Options/DeprecatedOptions.js

@@ -100,7 +100,8 @@ function deprecateFromOptionsMap(chart, rootOldAsArray, rootNewAsArray, mapToNew
         if (typeof val !== 'undefined') {
             traverseSetOption(rootNew, mapToNewOptions[oldOptionKey], val);
             error(32, false, chart, (_a = {},
-                _a[rootOldAsArray.join('.') + "." + oldOptionKey] = rootNewAsArray.join('.') + "." + mapToNewOptions[oldOptionKey].join('.'),
+                _a[rootOldAsArray.join('.') + '.' + oldOptionKey] = (rootNewAsArray.join('.') + '.' +
+                    mapToNewOptions[oldOptionKey].join('.')),
                 _a));
         }
     });
@@ -127,7 +128,9 @@ function copyDeprecatedAxisOptions(chart) {
         if (opts && opts.description) {
             opts.accessibility = opts.accessibility || {};
             opts.accessibility.description = opts.description;
-            error(32, false, chart, { 'axis.description': 'use axis.accessibility.description' });
+            error(32, false, chart, {
+                'axis.description': 'use axis.accessibility.description'
+            });
         }
     });
 }
@@ -157,8 +160,9 @@ function copyDeprecatedSeriesOptions(chart) {
             var optionVal = series.options[oldOption];
             // Special case
             if (oldOption === 'accessibility.pointDescriptionFormatter') {
-                optionVal = series.options.accessibility &&
-                    series.options.accessibility.pointDescriptionFormatter;
+                optionVal = (series.options.accessibility &&
+                    series.options.accessibility
+                        .pointDescriptionFormatter);
             }
             if (typeof optionVal !== 'undefined') {
                 // Set the new option
@@ -167,7 +171,10 @@ function copyDeprecatedSeriesOptions(chart) {
                 // value, since we set enabled rather than disabled
                 oldOption === 'skipKeyboardNavigation' ?
                     !optionVal : optionVal);
-                error(32, false, chart, (_a = {}, _a["series." + oldOption] = "series." + oldToNewSeriesOptions[oldOption].join('.'), _a));
+                error(32, false, chart, (_a = {},
+                    _a["series." + oldOption] = ('series.' +
+                        oldToNewSeriesOptions[oldOption].join('.')),
+                    _a));
             }
         });
     });

package/es-modules/Accessibility/Options/LangOptions.js

@@ -77,9 +77,9 @@ var langOptions = {
              */
             annotations: {
                 heading: 'Chart annotations summary',
-                descriptionSinglePoint: '{annotationText}. Related to {annotationPoint}',
-                descriptionMultiplePoints: '{annotationText}. Related to {annotationPoint}' +
-                    '{ Also related to, #each(additionalAnnotationPoints)}',
+                descriptionSinglePoint: ('{annotationText}. Related to {annotationPoint}'),
+                descriptionMultiplePoints: ('{annotationText}. Related to {annotationPoint}' +
+                    '{ Also related to, #each(additionalAnnotationPoints)}'),
                 descriptionNoPoints: '{annotationText}'
             },
             /**

package/es-modules/Accessibility/ProxyElement.js

@@ -45,13 +45,17 @@ var ProxyElement = /** @class */ (function () {
         this.chart = chart;
         this.target = target;
         this.groupType = groupType;
+        var isListItem = groupType === 'ul';
         this.eventProvider = new EventProvider();
-        var wrapperEl = groupType === 'ul' ? doc.createElement('li') : null;
+        var wrapperEl = isListItem ? doc.createElement('li') : null;
         var btnEl = this.buttonElement = doc.createElement('button');
         if (!chart.styledMode) {
             this.hideButtonVisually(btnEl);
         }
         if (wrapperEl) {
+            if (isListItem && !chart.styledMode) {
+                wrapperEl.style.listStyle = 'none';
+            }
             wrapperEl.appendChild(btnEl);
             this.element = wrapperEl;
         }
@@ -77,17 +81,24 @@ var ProxyElement = /** @class */ (function () {
         fireEventOnWrappedOrUnwrappedElement(this.target.click, fakeEventObject);
     };
     /**
-     * Update the target to be proxied.
-     * The position and events are updated to match the new target.
+     * Update the target to be proxied. The position and events are updated to
+     * match the new target.
      * @param target The new target definition
-     * @param attributes New HTML attributes to apply to the button. Set an attribute to null to remove.
+     * @param attributes New HTML attributes to apply to the button. Set an
+     * attribute to null to remove.
      */
     ProxyElement.prototype.updateTarget = function (target, attributes) {
         this.target = target;
         this.updateCSSClassName();
+        var attrs = attributes || {};
+        Object.keys(attrs).forEach(function (a) {
+            if (attrs[a] === null) {
+                delete attrs[a];
+            }
+        });
         attr(this.buttonElement, merge({
             'aria-label': this.getTargetAttr(target.click, 'aria-label')
-        }, attributes));
+        }, attrs));
         this.eventProvider.removeAddedEvents();
         this.addProxyEventsToButton(this.buttonElement, target.click);
         this.refreshPosition();
@@ -116,7 +127,7 @@ var ProxyElement = /** @class */ (function () {
      * Update the CSS class name to match target
      */
     ProxyElement.prototype.updateCSSClassName = function () {
-        var stringHasNoTooltip = function (s) { return s.indexOf('highcharts-no-tooltip') > -1; };
+        var stringHasNoTooltip = function (s) { return (s.indexOf('highcharts-no-tooltip') > -1); };
         var legend = this.chart.legend;
         var groupDiv = legend.group && legend.group.div;
         var noTooltipOnGroup = stringHasNoTooltip(groupDiv && groupDiv.className || '');
@@ -144,7 +155,8 @@ var ProxyElement = /** @class */ (function () {
                     fireEventOnWrappedOrUnwrappedElement(target, clonedEvent);
                 }
                 e.stopPropagation();
-                // #9682, #15318: Touch scrolling didnt work when touching proxy
+                // #9682, #15318: Touch scrolling didnt work when touching
+                // proxy
                 if (!isTouchEvent) {
                     e.preventDefault();
                 }
@@ -183,12 +195,14 @@ var ProxyElement = /** @class */ (function () {
         var posElement = this.target.visual || clickTargetElement;
         var chartDiv = this.chart.renderTo;
         if (chartDiv && posElement && posElement.getBoundingClientRect) {
-            var rectEl = posElement.getBoundingClientRect(), rectDiv = chartDiv.getBoundingClientRect();
+            var rectEl = posElement.getBoundingClientRect(), chartPos = this.chart.pointer.getChartPosition();
             return {
-                x: rectEl.left - rectDiv.left,
-                y: rectEl.top - rectDiv.top,
-                width: rectEl.right - rectEl.left,
-                height: rectEl.bottom - rectEl.top
+                x: (rectEl.left - chartPos.left) / chartPos.scaleX,
+                y: (rectEl.top - chartPos.top) / chartPos.scaleY,
+                width: rectEl.right / chartPos.scaleX -
+                    rectEl.left / chartPos.scaleX,
+                height: rectEl.bottom / chartPos.scaleY -
+                    rectEl.top / chartPos.scaleY
             };
         }
         return { x: 0, y: 0, width: 1, height: 1 };

package/es-modules/Accessibility/ProxyProvider.js

@@ -72,10 +72,13 @@ var ProxyProvider = /** @class */ (function () {
     /**
      * Create a group that will contain proxy elements. The group order is
      * automatically updated according to the last group order keys.
+     *
+     * Returns the added group.
      */
     ProxyProvider.prototype.addGroup = function (groupKey, groupType, attributes) {
-        if (this.groups[groupKey]) {
-            return;
+        var existingGroup = this.groups[groupKey];
+        if (existingGroup) {
+            return existingGroup.groupElement;
         }
         var proxyContainer = this.domElementProvider.createElement(groupType);
         // If we want to add a role to the group, and still use e.g.
@@ -98,15 +101,13 @@ var ProxyProvider = /** @class */ (function () {
         };
         attr(groupElement, attributes || {});
         if (groupType === 'ul') {
-            if (!this.chart.styledMode) {
-                proxyContainer.style.listStyle = 'none';
-            }
             proxyContainer.setAttribute('role', 'list'); // Needed for webkit
         }
         // Add the group to the end by default, and perhaps then we
         // won't have to reorder the whole set of groups.
         this.afterChartProxyPosContainer.appendChild(groupElement);
         this.updateGroupOrder(this.groupOrder);
+        return groupElement;
     };
     /**
      * Update HTML attributes of a group.
@@ -289,6 +290,10 @@ var ProxyProvider = /** @class */ (function () {
      */
     ProxyProvider.prototype.updatePosContainerPositions = function () {
         var chart = this.chart;
+        // If exporting, don't add these containers to the DOM.
+        if (chart.renderer.forExport) {
+            return;
+        }
         var rendererSVGEl = chart.renderer.box;
         chart.container.insertBefore(this.afterChartProxyPosContainer, rendererSVGEl.nextSibling);
         chart.container.insertBefore(this.beforeChartProxyPosContainer, rendererSVGEl);

package/es-modules/Accessibility/Utils/Announcer.js

@@ -51,12 +51,12 @@ var Announcer = /** @class */ (function () {
             clearTimeout(this.clearAnnouncementRegionTimer);
         }
         this.clearAnnouncementRegionTimer = setTimeout(function () {
-            _this.announceRegion.innerHTML = '';
+            _this.announceRegion.innerHTML = AST.emptyHTML;
             delete _this.clearAnnouncementRegionTimer;
         }, 1000);
     };
     Announcer.prototype.addAnnounceRegion = function (type) {
-        var chartContainer = this.chart.announcerContainer || this.createAnnouncerContainer(), div = this.domElementProvider.createElement('div');
+        var chartContainer = (this.chart.announcerContainer || this.createAnnouncerContainer()), div = this.domElementProvider.createElement('div');
         attr(div, {
             'aria-hidden': false,
             'aria-live': type

package/es-modules/Accessibility/Utils/ChartUtilities.js

@@ -24,12 +24,14 @@ var defined = U.defined, find = U.find, fireEvent = U.fireEvent;
 /* eslint-disable valid-jsdoc */
 /**
  * Fire an event on an element that is either wrapped by Highcharts,
- * or a DOM element
+ * or a DOM element.
+ * @private
  */
 function fireEventOnWrappedOrUnwrappedElement(el, eventObject) {
     var type = eventObject.type;
     var hcEvents = el.hcEvents;
-    if (doc.createEvent && (el.dispatchEvent || el.fireEvent)) {
+    if (doc.createEvent &&
+        (el.dispatchEvent || el.fireEvent)) {
         if (el.dispatchEvent) {
             el.dispatchEvent(eventObject);
         }
@@ -45,7 +47,7 @@ function fireEventOnWrappedOrUnwrappedElement(el, eventObject) {
     }
 }
 /**
- * @return {string}
+ * @private
  */
 function getChartTitle(chart) {
     return stripHTMLTags(chart.options.title.text ||
@@ -53,8 +55,7 @@ function getChartTitle(chart) {
 }
 /**
  * Return string with the axis name/title.
- * @param {Highcharts.Axis} axis
- * @return {string}
+ * @private
  */
 function getAxisDescription(axis) {
     return axis && (axis.userOptions && axis.userOptions.accessibility &&
@@ -67,8 +68,11 @@ function getAxisDescription(axis) {
 }
 /**
  * Return string with text description of the axis range.
- * @param {Highcharts.Axis} axis The axis to get range desc of.
- * @return {string} A string with the range description for the axis.
+ * @private
+ * @param {Highcharts.Axis} axis
+ * The axis to get range desc of.
+ * @return {string}
+ * A string with the range description for the axis.
  */
 function getAxisRangeDescription(axis) {
     var axisOptions = axis.options || {};
@@ -91,8 +95,7 @@ function getAxisRangeDescription(axis) {
 }
 /**
  * Describe the range of a category axis.
- * @param {Highcharts.Axis} axis
- * @return {string}
+ * @private
  */
 function getCategoryAxisRangeDesc(axis) {
     var chart = axis.chart;
@@ -107,8 +110,7 @@ function getCategoryAxisRangeDesc(axis) {
 }
 /**
  * Describe the length of the time window shown on an axis.
- * @param {Highcharts.Axis} axis
- * @return {string}
+ * @private
  */
 function getAxisTimeLengthDesc(axis) {
     var chart = axis.chart, range = {};
@@ -134,13 +136,12 @@ function getAxisTimeLengthDesc(axis) {
 }
 /**
  * Describe an axis from-to range.
- * @param {Highcharts.Axis} axis
- * @return {string}
+ * @private
  */
 function getAxisFromToDescription(axis) {
-    var chart = axis.chart, dateRangeFormat = (chart.options &&
-        chart.options.accessibility &&
-        chart.options.accessibility.screenReaderSection.axisRangeDateFormat ||
+    var chart = axis.chart, options = chart.options, dateRangeFormat = (options &&
+        options.accessibility &&
+        options.accessibility.screenReaderSection.axisRangeDateFormat ||
         ''), format = function (axisKey) {
         return axis.dateTime ? chart.time.dateFormat(dateRangeFormat, axis[axisKey]) : axis[axisKey];
     };
@@ -186,8 +187,6 @@ function getSeriesA11yElement(series) {
  * Remove aria-hidden from element. Also unhides parents of the element, and
  * hides siblings that are not explicitly unhidden.
  * @private
- * @param {Highcharts.Chart} chart
- * @param {Highcharts.HTMLDOMElement|Highcharts.SVGDOMElement} element
  */
 function unhideChartElementFromAT(chart, element) {
     element.setAttribute('aria-hidden', false);
@@ -209,9 +208,6 @@ function unhideChartElementFromAT(chart, element) {
 /**
  * Hide series from screen readers.
  * @private
- * @param {Highcharts.Series} series
- * The series to hide
- * @return {void}
  */
 function hideSeriesFromAT(series) {
     var seriesEl = getSeriesA11yElement(series);
@@ -222,9 +218,6 @@ function hideSeriesFromAT(series) {
 /**
  * Get series objects by series name.
  * @private
- * @param {Highcharts.Chart} chart
- * @param {string} name
- * @return {Array<Highcharts.Series>}
  */
 function getSeriesFromName(chart, name) {
     if (!name) {
@@ -237,10 +230,6 @@ function getSeriesFromName(chart, name) {
 /**
  * Get point in a series from x/y values.
  * @private
- * @param {Array<Highcharts.Series>} series
- * @param {number} x
- * @param {number} y
- * @return {Highcharts.Point|undefined}
  */
 function getPointFromXY(series, x, y) {
     var i = series.length, res;
@@ -256,9 +245,6 @@ function getPointFromXY(series, x, y) {
 /**
  * Get relative position of point on an x/y axis from 0 to 1.
  * @private
- * @param {Highcharts.Axis} axis
- * @param {Highcharts.Point} point
- * @return {number}
  */
 function getRelativePointAxisPosition(axis, point) {
     if (!defined(axis.dataMin) || !defined(axis.dataMax)) {
@@ -272,7 +258,6 @@ function getRelativePointAxisPosition(axis, point) {
 /**
  * Get relative position of point on an x/y axis from 0 to 1.
  * @private
- * @param {Highcharts.Point} point
  */
 function scrollToPoint(point) {
     var xAxis = point.series.xAxis, yAxis = point.series.yAxis, axis = (xAxis && xAxis.scrollbar ? xAxis : yAxis), scrollbar = (axis && axis.scrollbar);