Merge pull request #353 from plotly/zoom-on-line-bug

Correctly compute ultra-zoomed line paths [fixes #217]

Author: etpinard

src/plots/cartesian/set_convert.js

@@ -123,9 +123,9 @@ module.exports = function setConvert(ax) {
 
     ax.l2p = function(v) {
         if(!isNumeric(v)) return constants.BADNUM;
+
         // include 2 fractional digits on pixel, for PDF zooming etc
-        return d3.round(Lib.constrain(ax._b + ax._m*v,
-            -clipMult*ax._length, (1+clipMult)*ax._length), 2);
+        return d3.round(ax._b + ax._m * v, 2);
     };
 
     ax.p2l = function(px) { return (px-ax._b)/ax._m; };

test/image/baselines/ultra_zoom.png

@@ -0,0 +1,45 @@
+{
+    "data": [
+        {
+            "x": [
+                10000,
+                0.0001
+            ],
+            "y": [
+                0.0001,
+                10000
+            ],
+            "uid": "1376de"
+        }
+    ],
+    "layout": {
+        "xaxis": {
+            "range": [
+                4990.012238820174,
+                5009.888574816098
+            ],
+            "type": "linear",
+            "autorange": false
+        },
+        "yaxis": {
+            "range": [
+                4907.413431128477,
+                5132.099078595185
+            ],
+            "type": "linear",
+            "autorange": false
+        },
+        "shapes": [
+            {
+                "type": "line",
+                "x0": 10000,
+                "x1": 0.0001,
+                "y0": 10000,
+                "y1": 0.0001
+            }
+        ],
+        "height": 450,
+        "width": 1000,
+        "autosize": true
+    }
+}