codepen cleanup

Author: richardnm-2

src/plots/cartesian/axes.js

@@ -2504,9 +2504,8 @@ axes.drawOne = function(gd, ax, opts) {
     });
 
     var tickNames = ['tick'];
+
     if(ax.type === 'multicategory') {
-        // https://stackoverflow.com/questions/750486/javascript-closure-inside-loops-simple-practical-example
-        console.log('slice', ax.levels.slice().reverse().slice(0, ax.levelNr - 1));
         ax.levels.slice().reverse().slice(0, ax.levelNr - 1).forEach(function(_lvl) {
             var pad = {x: 0 * _lvl, y: 10}[axLetter];
 
@@ -2531,25 +2530,17 @@ axes.drawOne = function(gd, ax, opts) {
         });
 
         tickNames = tickNames.sort();
-        console.log('tickNames', tickNames);
 
         ax.levels.slice().forEach(function(_lvl, idx) {
             seq.push(function() {
                 ax._depth = (majorTickSigns[4] * (getLabelLevelBbox(tickNames.slice()[_lvl])[ax.side] - mainLinePosition));
-                console.log('depth', ax._depth);
-
-                // TODO Has to be removed to loop through all dividers?
-                // console.log('dividers', dividerVals);
 
                 var levelDividers = dividerVals.slice().filter(function(divider) {
                     return divider.level === idx;
                 });
 
-                console.log('levelDividers', levelDividers);
-
                 return drawDividers(gd, ax, {
                     vals: levelDividers,
-                    // vals: [],
                     layer: mainAxLayer,
                     path: axes.makeTickPath(ax, mainLinePosition, majorTickSigns[4], { len: ax._depth }),
                     transFn: transTickFn
@@ -3787,6 +3778,7 @@ function drawDividers(gd, ax, opts) {
     var dividers = opts.layer.selectAll('path.' + cls)
         .data(vals, tickDataFn);
 
+    // TODO Has to be removed to loop through all dividers??
     // dividers.exit().remove();
 
     dividers.enter().insert('path', ':first-child')

src/plots/cartesian/set_convert.js

@@ -390,11 +390,7 @@ module.exports = function setConvert(ax, fullLayout) {
 
                 if(axLetter in trace) {
                     var arrayIn = trace[axLetter];
-                    console.log(arrayIn);
-                    // var arrayIn = trace[axLetter].slice(0, 2);
 
-
-                    // if(isArrayOrTypedArray(arrayIn[0]) && isArrayOrTypedArray(arrayIn[1])) {
                     var arrays = arrayIn.map(function(x) {
                         return x;
                     });
@@ -404,6 +400,7 @@ module.exports = function setConvert(ax, fullLayout) {
 
                     Array.prototype.push.apply(fullObjectList, objList);
 
+                    // convert the trace data from list to object and sort (backwards, stable sort)
                     var sortedObjectList = sortLib.sortObjectList(cols, objList);
                     var matrix = sortLib.objectListToList(sortedObjectList);
                     var sortedMatrix = sortLib.sortedMatrix(matrix);
@@ -412,28 +409,21 @@ module.exports = function setConvert(ax, fullLayout) {
                     var y = sortedMatrix[1];
 
                     // Could/should set sorted y axis values for each trace as the sorted values are already available.
-                    // Need write access to gd._fullData
-
+                    // Need write access to gd._fullData, bad?
                     var transposedXs = sortLib.transpose(xs);
-                    // debugger;
                     gd._fullData[i].x = transposedXs;
                     gd._fullData[i].y = y;
-                    // console.log('trace', i);
-                    // console.log('gd', gd._fullData[i]);
-                    // }
                 }
             }
             ax.levelNr = xs[0].length;
             ax.levels = xs[0].map(function(_, idx) {return idx;});
 
 
-            console.log('fullObjectList', fullObjectList);
             var fullSortedObjectList = sortLib.sortObjectList(cols, fullObjectList.slice());
             var fullList = sortLib.objectListToList(fullSortedObjectList);
             var fullSortedMatrix = sortLib.sortedMatrix(fullList);
 
             var fullXs = fullSortedMatrix[0].slice();
-            console.log('fullXs', fullXs);
 
             for(i = 0; i < fullXs.length; i++) {
                 setCategoryIndex(fullXs[i]);