Fix NaN positions for shapes when dragging them.

Author: my-tien

src/components/shapes/draw_newshape/newshapes.js

@@ -67,6 +67,7 @@ function newShapes(outlines, dragOptions) {
     clearOutline(gd);
 
     var editHelpers = dragOptions.editHelpers;
+    var plotinfo = dragOptions.plotinfo;
     var modifyItem = (editHelpers || {}).modifyItem;
 
     var allShapes = [];
@@ -84,10 +85,21 @@ function newShapes(outlines, dragOptions) {
                 case 'line':
                 case 'rect':
                 case 'circle':
-                    modifyItem('x0', afterEdit.x0 - (beforeEdit.x0shift || 0));
-                    modifyItem('x1', afterEdit.x1 - (beforeEdit.x1shift || 0));
-                    modifyItem('y0', afterEdit.y0 - (beforeEdit.y0shift || 0));
-                    modifyItem('y1', afterEdit.y1 - (beforeEdit.y1shift || 0));
+                    if (beforeEdit.xref.includes("x") && plotinfo.xaxis.type.includes("category")) {
+                        plotinfo.xaxis.r2c(afterEdit.x0)
+                        modifyItem('x0', plotinfo.xaxis.r2c(afterEdit.x0) - (beforeEdit.x0shift || 0));
+                        modifyItem('x1', plotinfo.xaxis.r2c(afterEdit.x1) - (beforeEdit.x1shift || 0));
+                    } else {
+                        modifyItem('x0', afterEdit.x0);
+                        modifyItem('x1', afterEdit.x1);
+                    }
+                    if (beforeEdit.yref.includes("y") && plotinfo.yaxis.type.includes("category")) {
+                        modifyItem('y0', plotinfo.xaxis.r2c(afterEdit.y0) - (beforeEdit.y0shift || 0));
+                        modifyItem('y1', plotinfo.xaxis.r2c(afterEdit.y1) - (beforeEdit.y1shift || 0));
+                    } else {
+                        modifyItem('y0', afterEdit.y0);
+                        modifyItem('y1', afterEdit.y1);
+                    }
                     break;
 
                 case 'path':