Merge pull request #5415 from plotly/no-v2-changes

update comments about v2 changes that moved to another major

Author: archmoj

src/components/annotations/click.js

@@ -130,7 +130,7 @@ function getToggleSets(gd, hoverData) {
     return {on: onSet, off: offSet, explicitOff: explicitOffSet};
 }
 
-// to handle log axes until v2
+// to handle log axes until v3
 function clickData2r(d, ax) {
     return ax.type === 'log' ? ax.l2r(d) : ax.d2r(d);
 }

src/components/annotations/convert_coords.js

@@ -16,7 +16,7 @@ var toLogRange = require('../../lib/to_log_range');
  * convertCoords: when converting an axis between log and linear
  * you need to alter any annotations on that axis to keep them
  * pointing at the same data point.
- * In v2.0 this will become obsolete
+ * In v3.0 this will become obsolete
  *
  * gd: the plot div
  * ax: the axis being changed

src/components/colorscale/helpers.js

@@ -50,7 +50,7 @@ var letterAttrs = ['min', 'max', 'mid', 'auto'];
 /**
  * Extract 'c' / 'z', trace / color axis colorscale options
  *
- * Note that it would be nice to replace all z* with c* equivalents in v2
+ * Note that it would be nice to replace all z* with c* equivalents in v3
  *
  * @param {object} cont : attribute container
  * @return {object}:

src/components/images/convert_coords.js

@@ -16,7 +16,7 @@ var toLogRange = require('../../lib/to_log_range');
  * convertCoords: when converting an axis between log and linear
  * you need to alter any images on that axis to keep them
  * pointing at the same data point.
- * In v2.0 this will become obsolete (or perhaps size will still need conversion?)
+ * In v3.0 this will become obsolete (or perhaps size will still need conversion?)
  * we convert size by declaring that the maximum extent *in data units* should be
  * the same, assuming the image is anchored by its center (could remove that restriction
  * if we think it's important) even though the actual left and right values will not be

src/components/legend/defaults.js

@@ -99,7 +99,7 @@ module.exports = function legendDefaults(layoutIn, layoutOut, fullData) {
         } else {
             // maybe use y=1.1 / yanchor=bottom as above
             //   to avoid https://github.com/plotly/plotly.js/issues/1199
-            //   in v2
+            //   in v3
             defaultY = -0.1;
             defaultYAnchor = 'top';
         }

src/components/shapes/defaults.js

@@ -81,7 +81,7 @@ function handleShapeDefaults(shapeIn, shapeOut, fullLayout) {
             var dflt0 = 0.25;
             var dflt1 = 0.75;
 
-            // hack until V2.0 when log has regular range behavior - make it look like other
+            // hack until V3.0 when log has regular range behavior - make it look like other
             // ranges to send to coerce, then put it back after
             // this is all to give reasonable default position behavior on log axes, which is
             // a pretty unimportant edge case so we could just ignore this.

src/components/shapes/helpers.js

@@ -18,7 +18,7 @@ var Lib = require('../../lib');
 // so these have to be specified in terms of the category serial numbers,
 // but can take fractional values. Other axis types we specify position based on
 // the actual data values.
-// TODO: in V2.0 (when log axis ranges are in data units) range and shape position
+// TODO: in V3.0 (when log axis ranges are in data units) range and shape position
 // will be identical, so rangeToShapePosition and shapePositionToRange can be
 // removed entirely.
 

src/plot_api/plot_config.js

@@ -126,7 +126,7 @@ var configAttributes = {
         dflt: false,
         description: [
             'Determines whether to change the layout size when window is resized.',
-            'In v2, this option will be removed and will always be true.'
+            'In v3, this option will be removed and will always be true.'
         ].join(' ')
     },
     fillFrame: {

src/plots/cartesian/helpers.js

@@ -11,7 +11,7 @@
 
 var strTranslate = require('../../lib').strTranslate;
 
-// in v2 (once log ranges are fixed),
+// in v3 (once log ranges are fixed),
 // we'll be able to p2r here for all axis types
 function p2r(ax, v) {
     switch(ax.type) {

src/plots/cartesian/select.js

@@ -355,9 +355,9 @@ function prepSelect(e, startX, startY, dragOptions, mode) {
                 }
 
                 if(clickmode === 'event') {
-                    // TODO: remove in v2 - this was probably never intended to work as it does,
+                    // TODO: remove in v3 - this was probably never intended to work as it does,
                     // but in case anyone depends on it we don't want to break it now.
-                    // Note that click-to-select introduced pre v2 also emitts proper
+                    // Note that click-to-select introduced pre v3 also emitts proper
                     // event data when clickmode is having 'select' in its flag list.
                     gd.emit('plotly_selected', undefined);
                 }