add a few usefull TODOs

Author: etpinard

src/components/fx/hover.js

@@ -447,10 +447,12 @@ function _hover(gd, evt, subplot) {
             pointNumber: pt.index
         };
 
+        // could maybe :hocho: _module.eventData
         if(pt.trace._module.eventData) out = pt.trace._module.eventData(out, pt);
         else {
             out.x = pt.xVal;
             out.y = pt.yVal;
+
             out.xaxis = pt.xa;
             out.yaxis = pt.ya;
 

src/plots/plots.js

@@ -676,7 +676,9 @@ plots.supplyDataDefaults = function(dataIn, dataOut, layout, fullLayout) {
         Lib.pushUnique(modules, _module);
         Lib.pushUnique(basePlotModules, fullTrace._module.basePlotModule);
 
+        // TODO remove findArrayAttributes calls downstream
         fullTrace._arrayAttrs = PlotSchema.findArrayAttributes(fullTrace);
+
         cnt++;
     }
 

src/traces/scatter/select.js

@@ -47,6 +47,7 @@ module.exports = function selectPoints(searchInfo, polygon) {
                     pointNumber: i,
                     x: di.x,
                     y: di.y,
+                    // TODO generalize with hover/click data handler
                     id: di.id
                 });
                 di.dim = 0;