use p2r to show convert selection range evt data

... and p2d for log axes (until v2 !)

Author: etpinard

src/plots/cartesian/select.js

@@ -85,9 +85,15 @@ function prepSelect(e, startX, startY, dragOptions, mode) {
     var searchTraces = determineSearchTraces(gd, dragOptions.xaxes,
       dragOptions.yaxes, dragOptions.subplot);
 
+    // in v2 (once log ranges are fixed),
+    // we'll be able to p2r here for all axis types
+    function p2r(ax, v) {
+        return ax.type === 'log' ? ax.p2d(v) : ax.p2r(v);
+    }
+
     function axValue(ax) {
         var index = (ax._id.charAt(0) === 'y') ? 1 : 0;
-        return function(v) { return ax.p2d(v[index]); };
+        return function(v) { return p2r(ax, v[index]); };
     }
 
     function ascending(a, b) { return a - b; }
@@ -107,8 +113,8 @@ function prepSelect(e, startX, startY, dragOptions, mode) {
                     var axLetter = ax._id.charAt(0);
 
                     ranges[ax._id] = [
-                        ax.p2d(poly[axLetter + 'min']),
-                        ax.p2d(poly[axLetter + 'max'])
+                        p2r(ax, poly[axLetter + 'min']),
+                        p2r(ax, poly[axLetter + 'max'])
                     ].sort(ascending);
                 }
             };

test/jasmine/tests/select_test.js

@@ -1240,11 +1240,11 @@ describe('Test select box and lasso in general:', function() {
             rng: ['2000-01-20 19:48', '2000-04-06 01:48']
         }, {
             axType: 'category',
-            rng: [undefined, undefined]
+            rng: [-0.6208, 0.8375]
         }, {
             axType: 'multicategory',
-            rng: [undefined, undefined]
-        }]
+            rng: [-0.6208, 0.8375]
+        }];
 
         specs.forEach(function(s) {
             it('- @flaky on ' + s.axType + ' axes', function(done) {
@@ -1282,11 +1282,11 @@ describe('Test select box and lasso in general:', function() {
             pts: ['2000-12-25 21:36', '2000-12-28 22:48', '2001-01-01', '2001-01-01']
         }, {
             axType: 'category',
-            pts: [undefined, undefined, undefined, undefined]
+            pts: [5.8833, 5.9416, 6, 6]
         }, {
             axType: 'multicategory',
-            pts: [undefined, undefined, undefined, undefined]
-        }]
+            pts: [5.8833, 5.9416, 6, 6]
+        }];
 
         specs.forEach(function(s) {
             it('- @flaky on ' + s.axType + ' axes', function(done) {