revise dflt

Author: archmoj

src/plots/cartesian/axes.js

@@ -887,7 +887,7 @@ axes.calcTicks = function calcTicks(ax, opts) {
             value: x
         };
 
-        if(ticklabelstep && (id % (ticklabelstep + 1))) {
+        if(ticklabelstep > 1 && (id % ticklabelstep)) {
             obj.skipLabel = true;
         }
 
@@ -3007,7 +3007,7 @@ axes.drawLabels = function(gd, ax, opts) {
     var axLetter = axId.charAt(0);
     var cls = opts.cls || axId + 'tick';
 
-    var vals = ax.ticklabelstep ?
+    var vals = ax.ticklabelstep > 1 ?
         opts.vals.filter(function(d) { return !d.skipLabel; }) :
         opts.vals;
 

src/plots/cartesian/layout_attributes.js

@@ -414,14 +414,14 @@ module.exports = {
     },
     ticklabelstep: {
         valType: 'integer',
-        min: 0,
-        dflt: 0,
+        min: 1,
+        dflt: 1,
         editType: 'ticks',
         description: [
             'Sets the spacing between tick labels as compared to the spacing between ticks.',
-            'A value of 0 (default) means each tick gets a label.',
-            'A value of 1 means shows every 2nd level.',
-            'A larger value n means only every (n+1)th tick is labeled.',
+            'A value of 1 (default) means each tick gets a label.',
+            'A value of 2 means shows every 2nd level.',
+            'A larger value n means only every nth tick is labeled.',
             '`tick0` determines which labels are shown.'
         ].join(' ')
     },

test/image/mocks/h-colorbar05.json

@@ -19,7 +19,7 @@
           "len": 0.5,
           "ticks": "inside",
           "dtick": 20,
-          "ticklabelstep": 1,
+          "ticklabelstep": 2,
           "ticklabelposition": "inside right",
           "ticklen": 5,
           "bgcolor": "rgba(255,255,0,0.5)",
@@ -47,7 +47,7 @@
       }
     },
     "xaxis": {
-        "ticklabelstep": 1,
+        "ticklabelstep": 2,
         "side": "top"
     }
   }

test/image/mocks/indicator_attrs.json

@@ -108,7 +108,7 @@
                     "tickwidth": 5,
                     "tickcolor": "blue",
                     "showticklabels": true,
-                    "ticklabelstep": 1,
+                    "ticklabelstep": 2,
                     "tickfont": {
                         "family": "Arial",
                         "size": 16,

test/image/mocks/period_positioning7.json

@@ -381,7 +381,7 @@
             "dtick": 604800000,
             "tickformat": "%W",
             "ticklabelmode": "period",
-            "ticklabelstep": 3,
+            "ticklabelstep": 4,
             "tickcolor": "black"
         }
     }

test/image/mocks/polar_ticks.json

@@ -1854,7 +1854,7 @@
       },
       "radialaxis": {},
       "angularaxis": {
-        "ticklabelstep": 2,
+        "ticklabelstep": 3,
         "dtick": 10,
         "tickfont": {
           "size": 10,

test/image/mocks/ternary_array_styles.json

@@ -111,15 +111,15 @@
     "layout": {
         "ternary": {
             "aaxis": {
-                "ticklabelstep": 3,
+                "ticklabelstep": 4,
                 "dtick": 0.05,
                 "showline": true,
                 "ticks": "inside",
                 "showgrid": true,
                 "color": "#ccc"
             },
             "baxis": {
-                "ticklabelstep": 1,
+                "ticklabelstep": 2,
                 "dtick": 0.1,
                 "showline": true,
                 "ticks": "inside",

test/image/mocks/z-date_axes-ticklabelstep.json

@@ -73,19 +73,19 @@
     "showlegend": false,
     "width": 600,
     "height": 500,
-    "yaxis": {"ticklabelstep": 1, "domain": [0, 0.04]},
-    "yaxis2": {"ticklabelstep": 1, "domain": [0.16, 0.2]},
-    "yaxis3": {"ticklabelstep": 1, "domain": [0.32, 0.36]},
-    "yaxis4": {"ticklabelstep": 1, "domain": [0.48, 0.52]},
-    "yaxis5": {"ticklabelstep": 1, "domain": [0.64, 0.68]},
-    "yaxis6": {"ticklabelstep": 1, "domain": [0.80, 0.84]},
-    "yaxis7": {"ticklabelstep": 1, "domain": [0.96, 1]},
-    "xaxis": {"ticklabelstep": 1, "tickcolor": "black", "anchor": "y"},
-    "xaxis2": {"ticklabelstep": 1, "tickcolor": "black", "anchor": "y2"},
-    "xaxis3": {"ticklabelstep": 1, "tickcolor": "black", "anchor": "y3"},
-    "xaxis4": {"ticklabelstep": 1, "tickcolor": "black", "anchor": "y4"},
-    "xaxis5": {"ticklabelstep": 1, "tickcolor": "black", "anchor": "y5"},
-    "xaxis6": {"ticklabelstep": 1, "tickcolor": "black", "anchor": "y6"},
-    "xaxis7": {"ticklabelstep": 1, "tickcolor": "black", "anchor": "y7"}
+    "yaxis": {"ticklabelstep": 2, "domain": [0, 0.04]},
+    "yaxis2": {"ticklabelstep": 2, "domain": [0.16, 0.2]},
+    "yaxis3": {"ticklabelstep": 2, "domain": [0.32, 0.36]},
+    "yaxis4": {"ticklabelstep": 2, "domain": [0.48, 0.52]},
+    "yaxis5": {"ticklabelstep": 2, "domain": [0.64, 0.68]},
+    "yaxis6": {"ticklabelstep": 2, "domain": [0.80, 0.84]},
+    "yaxis7": {"ticklabelstep": 2, "domain": [0.96, 1]},
+    "xaxis": {"ticklabelstep": 2, "tickcolor": "black", "anchor": "y"},
+    "xaxis2": {"ticklabelstep": 2, "tickcolor": "black", "anchor": "y2"},
+    "xaxis3": {"ticklabelstep": 2, "tickcolor": "black", "anchor": "y3"},
+    "xaxis4": {"ticklabelstep": 2, "tickcolor": "black", "anchor": "y4"},
+    "xaxis5": {"ticklabelstep": 2, "tickcolor": "black", "anchor": "y5"},
+    "xaxis6": {"ticklabelstep": 2, "tickcolor": "black", "anchor": "y6"},
+    "xaxis7": {"ticklabelstep": 2, "tickcolor": "black", "anchor": "y7"}
   }
 }