plotly.js 2.12.0 → 2.12.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "plotly.js",
-  "version": "2.12.0",
+  "version": "2.12.1",
   "description": "The open source javascript graphing library that powers plotly",
   "license": "MIT",
   "main": "./lib/index.js",
@@ -127,30 +127,30 @@
     "browserify": "^17.0.0",
     "browserify-transform-tools": "^1.7.0",
     "bubleify": "^2.0.0",
-    "canvas": "^2.9.0",
+    "canvas": "^2.9.1",
     "check-node-version": "^4.2.1",
     "chttps": "^1.0.6",
     "deep-equal": "^2.0.5",
     "derequire": "^2.1.1",
     "ecstatic": "^4.1.4",
-    "eslint": "^8.10.0",
+    "eslint": "^8.14.0",
     "extra-iterable": "^2.5.22",
     "falafel": "^2.2.4",
-    "fs-extra": "^10.0.1",
-    "fuse.js": "^6.5.3",
+    "fs-extra": "^10.1.0",
+    "fuse.js": "^6.6.1",
     "glob": "^7.2.0",
     "gzip-size": "^6.0.0",
     "into-stream": "^6.0.0",
     "jasmine-core": "^3.5.0",
     "jsdom": "^19.0.0",
-    "karma": "^6.3.17",
+    "karma": "^6.3.19",
     "karma-browserify": "^8.1.0",
-    "karma-chrome-launcher": "^3.1.0",
+    "karma-chrome-launcher": "^3.1.1",
     "karma-firefox-launcher": "^2.1.2",
     "karma-ie-launcher": "^1.0.0",
     "karma-jasmine": "^3.3.1",
     "karma-jasmine-spec-tags": "^1.3.0",
-    "karma-spec-reporter": "^0.0.33",
+    "karma-spec-reporter": "^0.0.34",
     "karma-verbose-reporter": "^0.0.8",
     "karma-viewport": "1.0.2",
     "lodash": "^4.17.21",
@@ -160,13 +160,13 @@
     "minify-stream": "^2.1.0",
     "npm-link-check": "^4.0.0",
     "open": "^8.4.0",
-    "pixelmatch": "^5.2.1",
+    "pixelmatch": "^5.3.0",
     "prepend-file": "^2.0.1",
     "prettysize": "^2.0.0",
     "read-last-lines": "^1.8.0",
     "run-series": "^1.1.9",
     "sane-topojson": "^4.0.0",
-    "sass": "^1.49.9",
+    "sass": "^1.51.0",
     "through2": "^4.0.2",
     "true-case-path": "^2.2.1",
     "watchify": "^4.0.0"

package/src/components/colorscale/attributes.js

@@ -77,7 +77,7 @@ module.exports = function colorScaleAttrs(context, opts) {
     }
 
     var effectDesc = onlyIfNumerical ?
-        ' Has an effect only if ' + colorAttrFull + 'is set to a numerical array.' :
+        ' Has an effect only if ' + colorAttrFull + ' is set to a numerical array.' :
         '';
 
     var auto = cLetter + 'auto';
@@ -102,11 +102,11 @@ module.exports = function colorScaleAttrs(context, opts) {
             editType: editTypeOverride || 'style',
             description: [
                 'Sets the', context, 'color.',
-                ' It accepts either a specific color',
-                ' or an array of numbers that are mapped to the colorscale',
-                ' relative to the max and min values of the array or relative to',
-                ' ' + minmaxFull + ' if set.'
-            ].join('')
+                'It accepts either a specific color',
+                'or an array of numbers that are mapped to the colorscale',
+                'relative to the max and min values of the array or relative to',
+                minmaxFull, 'if set.'
+            ].join(' ')
         };
 
         if(opts.anim) {
@@ -121,11 +121,10 @@ module.exports = function colorScaleAttrs(context, opts) {
         impliedEdits: autoImpliedEdits,
         description: [
             'Determines whether or not the color domain is computed',
-            ' with respect to the input data (here ' + colorAttrFull + ') or the bounds set in',
-            ' ', minmaxFull,
-            ' ', effectDesc,
-            ' Defaults to `false` when ', minmaxFull, ' are set by the user.'
-        ].join('')
+            'with respect to the input data (here ' + colorAttrFull + ') or the bounds set in',
+            minmaxFull + effectDesc,
+            'Defaults to `false` when', minmaxFull, 'are set by the user.'
+        ].join(' ')
     };
 
     attrs[min] = {
@@ -134,11 +133,10 @@ module.exports = function colorScaleAttrs(context, opts) {
         editType: editTypeOverride || 'plot',
         impliedEdits: minmaxImpliedEdits,
         description: [
-            'Sets the lower bound of the color domain.',
-            effectDesc,
-            ' Value should have the same units as ', colorAttrFull,
-            ' and if set, ', maxFull, ' must be set as well.'
-        ].join('')
+            'Sets the lower bound of the color domain.' + effectDesc,
+            'Value should have the same units as', colorAttrFull,
+            'and if set,', maxFull, 'must be set as well.'
+        ].join(' ')
     };
 
     attrs[max] = {
@@ -147,11 +145,10 @@ module.exports = function colorScaleAttrs(context, opts) {
         editType: editTypeOverride || 'plot',
         impliedEdits: minmaxImpliedEdits,
         description: [
-            'Sets the upper bound of the color domain.',
-            effectDesc,
-            ' Value should have the same units as ', colorAttrFull,
-            ' and if set, ', minFull, ' must be set as well.'
-        ].join('')
+            'Sets the upper bound of the color domain.' + effectDesc,
+            'Value should have the same units as', colorAttrFull,
+            'and if set,', minFull, 'must be set as well.'
+        ].join(' ')
     };
 
     attrs[mid] = {
@@ -160,12 +157,11 @@ module.exports = function colorScaleAttrs(context, opts) {
         editType: 'calc',
         impliedEdits: autoImpliedEdits,
         description: [
-            'Sets the mid-point of the color domain by scaling ', minFull,
-            ' and/or ', maxFull, ' to be equidistant to this point.',
-            effectDesc,
-            ' Value should have the same units as ', colorAttrFull, '. ',
-            'Has no effect when ', autoFull, ' is `false`.'
-        ].join('')
+            'Sets the mid-point of the color domain by scaling', minFull,
+            'and/or', maxFull, 'to be equidistant to this point.' + effectDesc,
+            'Value should have the same units as', colorAttrFull + '.',
+            'Has no effect when', autoFull, 'is `false`.'
+        ].join(' ')
     };
 
     attrs.colorscale = {
@@ -174,19 +170,18 @@ module.exports = function colorScaleAttrs(context, opts) {
         dflt: colorscaleDflt,
         impliedEdits: {autocolorscale: false},
         description: [
-            'Sets the colorscale.',
-            effectDesc,
-            ' The colorscale must be an array containing',
-            ' arrays mapping a normalized value to an',
-            ' rgb, rgba, hex, hsl, hsv, or named color string.',
-            ' At minimum, a mapping for the lowest (0) and highest (1)',
-            ' values are required. For example,',
-            ' `[[0, \'rgb(0,0,255)\'], [1, \'rgb(255,0,0)\']]`.',
-            ' To control the bounds of the colorscale in color space,',
-            ' use', minmaxFull, '.',
-            ' Alternatively, `colorscale` may be a palette name string',
-            ' of the following list: ' + paletteStr + '.'
-        ].join('')
+            'Sets the colorscale.' + effectDesc,
+            'The colorscale must be an array containing',
+            'arrays mapping a normalized value to an',
+            'rgb, rgba, hex, hsl, hsv, or named color string.',
+            'At minimum, a mapping for the lowest (0) and highest (1)',
+            'values are required. For example,',
+            '`[[0, \'rgb(0,0,255)\'], [1, \'rgb(255,0,0)\']]`.',
+            'To control the bounds of the colorscale in color space,',
+            'use', minmaxFull + '.',
+            'Alternatively, `colorscale` may be a palette name string',
+            'of the following list: ' + paletteStr + '.'
+        ].join(' ')
     };
 
     attrs.autocolorscale = {
@@ -197,12 +192,11 @@ module.exports = function colorScaleAttrs(context, opts) {
         impliedEdits: {colorscale: undefined},
         description: [
             'Determines whether the colorscale is a default palette (`autocolorscale: true`)',
-            ' or the palette determined by ', code(contextHead + 'colorscale'), '.',
-            effectDesc,
-            ' In case `colorscale` is unspecified or `autocolorscale` is true, the default ',
-            ' palette will be chosen according to whether numbers in the `color` array are',
-            ' all positive, all negative or mixed.'
-        ].join('')
+            'or the palette determined by', code(contextHead + 'colorscale') + '.' + effectDesc,
+            'In case `colorscale` is unspecified or `autocolorscale` is true, the default',
+            'palette will be chosen according to whether numbers in the `color` array are',
+            'all positive, all negative or mixed.'
+        ].join(' ')
     };
 
     attrs.reversescale = {
@@ -210,11 +204,10 @@ module.exports = function colorScaleAttrs(context, opts) {
         dflt: false,
         editType: 'plot',
         description: [
-            'Reverses the color mapping if true.',
-            effectDesc,
-            ' If true, ', minFull, ' will correspond to the last color',
-            ' in the array and ', maxFull, ' will correspond to the first color.'
-        ].join('')
+            'Reverses the color mapping if true.' + effectDesc,
+            'If true,', minFull, 'will correspond to the last color',
+            'in the array and', maxFull, 'will correspond to the first color.'
+        ].join(' ')
     };
 
     if(!noScale) {
@@ -223,9 +216,8 @@ module.exports = function colorScaleAttrs(context, opts) {
             dflt: showScaleDflt,
             editType: 'calc',
             description: [
-                'Determines whether or not a colorbar is displayed for this trace.',
-                effectDesc
-            ].join('')
+                'Determines whether or not a colorbar is displayed for this trace.' + effectDesc
+            ].join(' ')
         };
 
         attrs.colorbar = colorbarAttrs;

package/src/components/modebar/manage.js

@@ -8,6 +8,7 @@ var isUnifiedHover = require('../fx/helpers').isUnifiedHover;
 var createModeBar = require('./modebar');
 var modeBarButtons = require('./buttons');
 var DRAW_MODES = require('./constants').DRAW_MODES;
+var extendDeep = require('../../lib').extendDeep;
 
 /**
  * ModeBar wrapper around 'create' and 'update',
@@ -330,7 +331,9 @@ function appendButtonsToGroups(groups, buttons) {
 }
 
 // fill in custom buttons referring to default mode bar buttons
-function fillCustomButton(customButtons) {
+function fillCustomButton(originalModeBarButtons) {
+    var customButtons = extendDeep([], originalModeBarButtons);
+
     for(var i = 0; i < customButtons.length; i++) {
         var buttonGroup = customButtons[i];
 

package/src/plot_api/plot_api.js

@@ -2171,7 +2171,9 @@ function _relayout(gd, aobj) {
             if(parentFull.autorange) flags.calc = true;
             else flags.plot = true;
         } else {
-            if((fullLayout._has('scatter-like') && fullLayout._has('regl')) &&
+            if(ai === 'dragmode' && ((vi === false && vOld !== false) || (vi !== false && vOld === false))) {
+                flags.plot = true;
+            } else if((fullLayout._has('scatter-like') && fullLayout._has('regl')) &&
                 (ai === 'dragmode' &&
                 (vi === 'lasso' || vi === 'select') &&
                 !(vOld === 'lasso' || vOld === 'select'))

package/src/plots/geo/geo.js

@@ -119,7 +119,7 @@ proto.fetchTopojson = function() {
                 if(err.status === 404) {
                     return reject(new Error([
                         'plotly.js could not find topojson file at',
-                        topojsonPath, '.',
+                        topojsonPath + '.',
                         'Make sure the *topojsonURL* plot config option',
                         'is set properly.'
                     ].join(' ')));

package/src/plots/polar/polar.js

@@ -800,7 +800,7 @@ proto.updateHoverAndMainDrag = function(fullLayout) {
     var scaleX;
     var scaleY;
 
-    var mainDrag = dragBox.makeDragger(layers, 'path', 'maindrag', 'crosshair');
+    var mainDrag = dragBox.makeDragger(layers, 'path', 'maindrag', fullLayout.dragmode === false ? 'none' : 'crosshair');
 
     d3.select(mainDrag)
         .attr('d', _this.pathSubplot())
@@ -1150,6 +1150,9 @@ proto.updateRadialDrag = function(fullLayout, polarLayout, rngIndex) {
 
     var radialDrag = dragBox.makeRectDragger(layers, className, 'crosshair', -bl2, -bl2, bl, bl);
     var dragOpts = {element: radialDrag, gd: gd};
+    if(fullLayout.dragmode === false) {
+        dragOpts.dragmode = false;
+    }
 
     updateElement(d3.select(radialDrag), radialAxis.visible && innerRadius < radius, {
         transform: strTranslate(tx, ty)
@@ -1292,13 +1295,17 @@ proto.updateAngularDrag = function(fullLayout) {
     var cyy = _this.cyy;
     var dbs = constants.angularDragBoxSize;
 
-    var angularDrag = dragBox.makeDragger(layers, 'path', 'angulardrag', 'move');
+    var angularDrag = dragBox.makeDragger(layers, 'path', 'angulardrag', fullLayout.dragmode === false ? 'none' : 'move');
     var dragOpts = {element: angularDrag, gd: gd};
 
-    d3.select(angularDrag)
-        .attr('d', _this.pathAnnulus(radius, radius + dbs))
-        .attr('transform', strTranslate(cx, cy))
-        .call(setCursor, 'move');
+    if(fullLayout.dragmode === false) {
+        dragOpts.dragmode = false;
+    } else {
+        d3.select(angularDrag)
+            .attr('d', _this.pathAnnulus(radius, radius + dbs))
+            .attr('transform', strTranslate(cx, cy))
+            .call(setCursor, 'move');
+    }
 
     function xy2a(x, y) {
         return Math.atan2(cyy + dbs - y, x - cxx - dbs);

package/src/registry.js

@@ -276,7 +276,7 @@ function registerTraceModule(_module) {
     // add `PlotlyGeoAssets` global to stash references to all fetched
     // topojson / geojson data
     if((bpmName === 'geo' || bpmName === 'mapbox') &&
-        (typeof window !== undefined && window.PlotlyGeoAssets === undefined)
+        (window.PlotlyGeoAssets === undefined)
     ) {
         window.PlotlyGeoAssets = {topojson: {}};
     }

package/src/traces/pointcloud/attributes.js

@@ -53,7 +53,7 @@ module.exports = {
                 'Sets the marker fill color. It accepts a specific color.',
                 'If the color is not fully opaque and there are hundreds of thousands',
                 'of points, it may cause slower zooming and panning.'
-            ].join('')
+            ].join(' ')
         },
         opacity: {
             valType: 'number',

package/src/traces/sankey/attributes.js

@@ -249,17 +249,13 @@ var attrs = module.exports = overrideAll({
                 valType: 'number',
                 editType: 'calc',
                 dflt: 1,
-                description: [
-                    'Sets the upper bound of the color domain.'
-                ].join('')
+                description: 'Sets the upper bound of the color domain.'
             },
             cmin: {
                 valType: 'number',
                 editType: 'calc',
                 dflt: 0,
-                description: [
-                    'Sets the lower bound of the color domain.'
-                ].join('')
+                description: 'Sets the lower bound of the color domain.'
             },
             colorscale: extendFlat(colorAttributes().colorscale, {dflt: [[0, 'white'], [1, 'black']]})
         }),

package/src/traces/surface/attributes.js

@@ -276,16 +276,16 @@ colorScaleAttrs('', {
         editType: 'calc',
         description: [
             'Sets the opacityscale.',
-            ' The opacityscale must be an array containing',
-            ' arrays mapping a normalized value to an opacity value.',
-            ' At minimum, a mapping for the lowest (0) and highest (1)',
-            ' values are required. For example,',
-            ' `[[0, 1], [0.5, 0.2], [1, 1]]` means that higher/lower values would have',
-            ' higher opacity values and those in the middle would be more transparent',
-            ' Alternatively, `opacityscale` may be a palette name string',
-            ' of the following list: \'min\', \'max\', \'extremes\' and \'uniform\'.',
-            ' The default is \'uniform\'.'
-        ].join('')
+            'The opacityscale must be an array containing',
+            'arrays mapping a normalized value to an opacity value.',
+            'At minimum, a mapping for the lowest (0) and highest (1)',
+            'values are required. For example,',
+            '`[[0, 1], [0.5, 0.2], [1, 1]]` means that higher/lower values would have',
+            'higher opacity values and those in the middle would be more transparent',
+            'Alternatively, `opacityscale` may be a palette name string',
+            'of the following list: \'min\', \'max\', \'extremes\' and \'uniform\'.',
+            'The default is \'uniform\'.'
+        ].join(' ')
     },
 
     _deprecated: {

package/src/traces/table/attributes.js

@@ -89,8 +89,8 @@ var attrs = module.exports = overrideAll({
                 dflt: 'white',
                 description: [
                     'Sets the cell fill color. It accepts either a specific color',
-                    ' or an array of colors or a 2D array of colors.'
-                ].join('')
+                    'or an array of colors or a 2D array of colors.'
+                ].join(' ')
             }
         },
 
@@ -157,8 +157,8 @@ var attrs = module.exports = overrideAll({
                 dflt: 'white',
                 description: [
                     'Sets the cell fill color. It accepts either a specific color',
-                    ' or an array of colors or a 2D array of colors.'
-                ].join('')
+                    'or an array of colors or a 2D array of colors.'
+                ].join(' ')
             }
         },
 

package/src/traces/violin/attributes.js

@@ -64,7 +64,7 @@ module.exports = {
             '*width* means each violin has the same (max) width',
             '*count* means the violins are scaled by the number of sample points making',
             'up each violin.'
-        ].join('')
+        ].join(' ')
     },
 
     spanmode: {

package/src/version.js

@@ -1,4 +1,4 @@
 'use strict';
 
 // package version injected by `npm run preprocess`
-exports.version = '2.12.0';
+exports.version = '2.12.1';