reverse loop order

Author: archmoj

src/plots/cartesian/axes.js

@@ -635,9 +635,9 @@ axes.calcTicks = function calcTicks(ax) {
             var newTickVals = [];
             var prevPos;
 
-            var dir = axrev ? -1 : 1;
-            var first = axrev ? len - 1 : 0;
-            var last = axrev ? 0 : len - 1;
+            var dir = axrev ? 1 : -1;
+            var first = axrev ? 0 : len - 1;
+            var last = axrev ? len - 1 : 0;
             for(var q = first; dir * q <= dir * last; q += dir) {
                 var tickVal = tickVals[q];
                 if(ax.maskBreaks(tickVal.value) === BADNUM) {
@@ -660,7 +660,7 @@ axes.calcTicks = function calcTicks(ax) {
                     newTickVals.push(tickVal);
                 }
             }
-            tickVals = newTickVals;
+            tickVals = newTickVals.reverse();
         }
     }
 

test/image/baselines/axes_breaks-dtick_auto.png

@@ -5016,7 +5016,7 @@ describe('Test axes', function() {
                 })
                 .then(function() {
                     _assert('with two rangebreaks', {
-                        tickVals: [0, 89, 100, 200]
+                        tickVals: [0, 89, 189, 200]
                     });
                 })
                 .catch(failTest)