add comment about smarter surface showscale default

etpinard · etpinard · commit ab2b9e80baee · 2016-04-21T11:52:19.000-04:00
diff --git a/src/traces/surface/defaults.js b/src/traces/surface/defaults.js
@@ -94,6 +94,9 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout
         mapLegacy(traceIn, 'zauto', 'cauto');
     }
 
+    // TODO if contours.?.usecolormap are false and hidesurface is true
+    // the colorbar shouldn't be shown by default
+
     colorscaleDefaults(
         traceIn, traceOut, layout, coerce, {prefix: '', cLetter: 'c'}
     );

Original file line number	Diff line number	Diff line change
`@@ -94,6 +94,9 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout`
`94`	`94`	`mapLegacy(traceIn, 'zauto', 'cauto');`
`95`	`95`	`}`
`96`	`96`
	`97`	`+ // TODO if contours.?.usecolormap are false and hidesurface is true`
	`98`	`+ // the colorbar shouldn't be shown by default`
	`99`	`+`
`97`	`100`	`colorscaleDefaults(`
`98`	`101`	`traceIn, traceOut, layout, coerce, {prefix: '', cLetter: 'c'}`
`99`	`102`	`);`