Merge pull request #5349 from jbampton/fix-spelling

Fix spelling

src/components/colorscale/index.js

@@ -25,7 +25,7 @@ module.exports = {
     calc: require('./calc'),
 
     // ./scales.js is required in lib/coerce.js ;
-    // it needs to be a seperate module to avoid circular a dependency
+    // it needs to be a separate module to avoid circular a dependency
     scales: scales.scales,
     defaultScale: scales.defaultScale,
     getScale: scales.get,

src/components/errorbars/attributes.js

@@ -32,7 +32,7 @@ module.exports = {
             'If *percent*, the bar lengths correspond to a percentage of',
             'underlying data. Set this percentage in `value`.',
 
-            'If *sqrt*, the bar lengths correspond to the sqaure of the',
+            'If *sqrt*, the bar lengths correspond to the square of the',
             'underlying data.',
 
             'If *data*, the bar lengths are set with data set `array`.'

src/components/fx/hover.js

@@ -1306,7 +1306,7 @@ function getHoverLabelText(d, showCommonLabel, hovermode, fullLayout, t0, g) {
 
 // Make groups of touching points, and within each group
 // move each point so that no labels overlap, but the average
-// label position is the same as it was before moving. Indicentally,
+// label position is the same as it was before moving. Incidentally,
 // this is equivalent to saying all the labels are on equal linear
 // springs about their initial position. Initially, each point is
 // its own group, but as we find overlaps we will clump the points.

src/components/sliders/draw.js

@@ -571,7 +571,7 @@ function setGripPosition(sliderGroup, sliderOpts, doTransition) {
             .ease(sliderOpts.transition.easing);
     }
 
-    // Drawing.setTranslate doesn't work here becasue of the transition duck-typing.
+    // Drawing.setTranslate doesn't work here because of the transition duck-typing.
     // It's also not necessary because there are no other transitions to preserve.
     el.attr('transform', strTranslate(x - constants.gripWidth * 0.5, sliderOpts._dims.currentValueTotalHeight));
 }

src/lib/index.js

@@ -865,7 +865,7 @@ lib.objectFromPath = function(path, value) {
  *   lib.expandObjectPaths({'foo[1].bar': 10, 'foo[0].bar': 20});
  *     => { foo: [{bar: 10}, {bar: 20}] }
  *
- * It does NOT, however, merge mulitple mutliply-nested arrays::
+ * It does NOT, however, merge multiple multiply-nested arrays::
  *
  *   lib.expandObjectPaths({'marker[1].range[1]': 5, 'marker[1].range[0]': 4})
  *     => { marker: [null, {range: 4}] }

src/plot_api/plot_api.js

@@ -3231,7 +3231,7 @@ function animate(gd, frameOrGroupNameOrFrameList, animationOpts) {
     }
 
     // Execute a callback after the wrapper function has been called n times.
-    // This is used to defer the resolution until a transition has resovled *and*
+    // This is used to defer the resolution until a transition has resolved *and*
     // the frame has completed. If it's not done this way, then we get a race
     // condition in which the animation might resolve before a transition is complete
     // or vice versa.

src/traces/carpet/attributes.js

@@ -37,7 +37,7 @@ module.exports = {
         editType: 'calc+clearAxisTypes',
         description: [
             'A two dimensional array of x coordinates at each carpet point.',
-            'If ommitted, the plot is a cheater plot and the xaxis is hidden',
+            'If omitted, the plot is a cheater plot and the xaxis is hidden',
             'by default.'
         ].join(' ')
     },
@@ -109,7 +109,7 @@ module.exports = {
         editType: 'calc',
         description: [
             'The shift applied to each successive row of data in creating a cheater plot.',
-            'Only used if `x` is been ommitted.'
+            'Only used if `x` is been omitted.'
         ].join(' ')
     },
     aaxis: axisAttrs,

src/traces/carpet/create_i_derivative_evaluator.js

@@ -19,7 +19,7 @@
  * Also note that the discontinuity of the derivative is in magnitude only. The direction *is*
  * continuous across cell boundaries.
  *
- * For example, to compute the derivative of the xcoordinate halfway betwen the 7 and 8th i-gridpoints
+ * For example, to compute the derivative of the xcoordinate halfway between the 7 and 8th i-gridpoints
  * and the 10th and 11th j-gridpoints given bicubic smoothing in both dimensions, you'd write:
  *
  *     var deriv = createIDerivativeEvaluator([x], 1, 1);

src/traces/carpet/plot.js

@@ -45,7 +45,7 @@ module.exports = function plot(gd, plotinfo, cdcarpet, carpetLayer) {
         drawGridLines(xa, ya, minorLayer, aax, 'a', aax._minorgridlines, true);
         drawGridLines(xa, ya, minorLayer, bax, 'b', bax._minorgridlines, true);
 
-        // NB: These are not ommitted if the lines are not active. The joins must be executed
+        // NB: These are not omitted if the lines are not active. The joins must be executed
         // in order for them to get cleaned up without a full redraw
         drawGridLines(xa, ya, boundaryLayer, aax, 'a-boundary', aax._boundarylines);
         drawGridLines(xa, ya, boundaryLayer, bax, 'b-boundary', bax._boundarylines);

src/traces/carpet/set_convert.js

@@ -215,15 +215,15 @@ module.exports = function setConvert(trace) {
 
         var i0 = Math.max(0, Math.min(a.length - 2, i));
 
-        // The step (demoninator) is implicitly 1 since that's the grid spacing.
+        // The step (denominator) is implicitly 1 since that's the grid spacing.
         return a[i0 + 1] - a[i0];
     };
 
     trace.dbdj = function(j /* , v*/) {
         // See above caveats for dadi which also apply here
         var j0 = Math.max(0, Math.min(b.length - 2, j));
 
-        // The step (demoninator) is implicitly 1 since that's the grid spacing.
+        // The step (denominator) is implicitly 1 since that's the grid spacing.
         return b[j0 + 1] - b[j0];
     };
 

src/traces/carpet/smooth_fill_2d_array.js

@@ -200,7 +200,7 @@ module.exports = function smoothFill2dArray(data, a, b) {
             overrelaxation = boundaryCnt ? 0 : 0.85;
 
             // If there are four non-null neighbors, then we want a simple average without
-            // overrelaxation. If all the surrouding points are null, then we want the full
+            // overrelaxation. If all the surrounding points are null, then we want the full
             // overrelaxation
             //
             // Based on experiments, this actually seems to slow down convergence just a bit.

src/traces/parcats/attributes.js

@@ -75,7 +75,7 @@ module.exports = {
         ],
         description: [
             'This value here applies when hovering over dimensions.',
-            'Note tath `*categorycount`, *colorcount* and *bandcolorcount*',
+            'Note that `*categorycount`, *colorcount* and *bandcolorcount*',
             'are only available when `hoveron` contains the *color* flag'
         ].join(' ')
     }),

src/traces/scatter/plot.js

@@ -491,7 +491,7 @@ function plotOne(gd, idx, plotinfo, cdscatter, cdscatterAll, element, transition
         join.selectAll('text')
             .call(Drawing.textPointStyle, trace, gd)
             .each(function(d) {
-                // This just *has* to be totally custom becuase of SVG text positioning :(
+                // This just *has* to be totally custom because of SVG text positioning :(
                 // It's obviously copied from translatePoint; we just can't use that
                 var x = xa.c2p(d.x);
                 var y = ya.c2p(d.y);

src/traces/scattercarpet/plot.js

@@ -30,7 +30,7 @@ module.exports = function plot(gd, plotinfoproxy, data, layer) {
         trace = data[i][0].trace;
 
         // Note: .select is adequate but seems to mutate the node data,
-        // which is at least a bit suprising and causes problems elsewhere
+        // which is at least a bit surprising and causes problems elsewhere
         node = layer.selectAll('g.trace' + trace.uid + ' .js-line');
 
         // Note: it would be more efficient if this didn't need to be applied

test/jasmine/assets/domain_ref/components.js

@@ -219,7 +219,7 @@ function annaxscale(ac, c0) {
 }
 
 // This tests to see that an annotation was drawn correctly.
-// Determinining the length of the arrow seems complicated due to the
+// Determining the length of the arrow seems complicated due to the
 // rectangle containing the text, so we draw 2 annotations, one K times the
 // length of the other, and solve for the desired arrow length from the
 // length measured on the screen. This works because multiplying the length

test/jasmine/tests/cartesian_test.js

@@ -574,7 +574,7 @@ describe('subplot creation / deletion:', function() {
             return Plotly.relayout(gd, {'xaxis2.domain': [0.49, 1]});
         })
         .then(function() {
-            // need to draw one backgroud <rect>
+            // need to draw one background <rect>
             checkBGLayers(0, 1, ['xy', 'x2y2', 'xy3']);
 
             // x ranges overlap, but now y ranges are disjoint
@@ -593,14 +593,14 @@ describe('subplot creation / deletion:', function() {
             });
         })
         .then(function() {
-            // need to draw one backgroud <rect>
+            // need to draw one background <rect>
             checkBGLayers(0, 1, ['xy', 'x2y2', 'xy3']);
 
             // change paper color
             return Plotly.relayout(gd, 'paper_bgcolor', 'black');
         })
         .then(function() {
-            // need a backgroud <rect> on main subplot to distinguish plot from
+            // need a background <rect> on main subplot to distinguish plot from
             // paper color
             checkBGLayers(1, 1, ['xy', 'x2y2', 'xy3']);
 

test/jasmine/tests/hover_label_test.js

@@ -4351,7 +4351,7 @@ describe('hover label rotation:', function() {
         });
     });
 
-    describe('when mulitple pts are picked', function() {
+    describe('when multiple pts are picked', function() {
         afterAll(destroyGraphDiv);
 
         beforeAll(function(done) {

test/jasmine/tests/lib_test.js

@@ -654,13 +654,13 @@ describe('Test lib.js:', function() {
         //
         //   // => {marker: {range: [null, 2]}}
         //
-        // This case *does* work becuase the array merging does not require a deep extend:
+        // This case *does* work because the array merging does not require a deep extend:
         //
         //   Lib.expandObjectPaths({'range[0]': 5, 'range[1]': 2}
         //
         //   // => {range: [5, 2]}
         //
-        // Finally note that this case works fine becuase there's no merge necessary:
+        // Finally note that this case works fine because there's no merge necessary:
         //
         //   Lib.expandObjectPaths({'marker.range[1]': 2})
         //

test/jasmine/tests/mock_test.js

@@ -724,8 +724,8 @@ var list = [
     'mapbox_choropleth0-legend',
     'mapbox_connectgaps',
     'mapbox_custom-style',
-    'mapbox_density-mulitple',
-    'mapbox_density-mulitple_legend',
+    'mapbox_density-multiple',
+    'mapbox_density-multiple_legend',
     'mapbox_density0',
     'mapbox_density0-legend',
     'mapbox_earthquake-density',
@@ -1815,8 +1815,8 @@ figs['mapbox_choropleth0'] = require('@mocks/mapbox_choropleth0');
 figs['mapbox_choropleth0-legend'] = require('@mocks/mapbox_choropleth0-legend');
 figs['mapbox_connectgaps'] = require('@mocks/mapbox_connectgaps');
 figs['mapbox_custom-style'] = require('@mocks/mapbox_custom-style');
-figs['mapbox_density-mulitple'] = require('@mocks/mapbox_density-mulitple');
-figs['mapbox_density-mulitple_legend'] = require('@mocks/mapbox_density-mulitple_legend');
+figs['mapbox_density-multiple'] = require('@mocks/mapbox_density-multiple');
+figs['mapbox_density-multiple_legend'] = require('@mocks/mapbox_density-multiple_legend');
 figs['mapbox_density0'] = require('@mocks/mapbox_density0');
 figs['mapbox_density0-legend'] = require('@mocks/mapbox_density0-legend');
 figs['mapbox_earthquake-density'] = require('@mocks/mapbox_earthquake-density');

test/jasmine/tests/range_slider_test.js

@@ -266,7 +266,7 @@ describe('Visible rangesliders', function() {
                 rangeslider: {visible: true}
             },
             width: 800,
-            hieght: 500
+            height: 500
         })
         .then(function() {
             var bb = getRangeSlider().getBoundingClientRect();

test/jasmine/tests/transform_groupby_test.js

@@ -143,7 +143,7 @@ describe('groupby', function() {
             .then(done);
 
             // The final test for restyle updates using deprecated syntax
-            // is ommitted since old style syntax is *only* sanitized on
+            // is omitted since old style syntax is *only* sanitized on
             // initial plot, *not* on restyle.
         });