add TODO comments to avoid casting to Array in supply defaults

Author: archmoj

src/plots/smith/layout_defaults.js

@@ -17,6 +17,7 @@ var constants = require('./constants');
 var axisNames = constants.axisNames;
 
 var makeImagDflt = memoize(function(realTickvals) {
+    // TODO: handle this case outside supply defaults step
     if(Lib.isTypedArray(realTickvals)) realTickvals = Array.from(realTickvals);
 
     return realTickvals.slice().reverse().map(function(x) { return -x; })
@@ -71,6 +72,7 @@ function handleDefaults(contIn, contOut, coerce, opts) {
                 coerceAxis('tickvals', imagTickvalsDflt);
             }
 
+            // TODO: handle this case outside supply defaults step
             if(Lib.isTypedArray(axOut.tickvals)) axOut.tickvals = Array.from(axOut.tickvals);
 
             var dfltColor;

src/traces/scatterpolar/defaults.js

@@ -71,6 +71,7 @@ function handleRThetaDefaults(traceIn, traceOut, layout, coerce) {
     var r = coerce('r');
     var theta = coerce('theta');
 
+    // TODO: handle this case outside supply defaults step
     if(Lib.isTypedArray(r)) {
         traceOut.r = r = Array.from(r);
     }

src/traces/scattersmith/defaults.js

@@ -75,6 +75,7 @@ function handleRealImagDefaults(traceIn, traceOut, layout, coerce) {
         len = Math.min(real.length, imag.length);
     }
 
+    // TODO: handle this case outside supply defaults step
     if(Lib.isTypedArray(real)) {
         traceOut.real = real = Array.from(real);
     }