Only coerce x0/x1/y0/y1shift if referenced axis is category/multicategory

Author: my-tien

src/components/selections/defaults.js

@@ -49,10 +49,6 @@ function handleSelectionDefaults(selectionIn, selectionOut, fullLayout) {
     coerce('line.color');
     coerce('line.width');
     coerce('line.dash');
-    coerce('x0shift');
-    coerce('x1shift');
-    coerce('y0shift');
-    coerce('y1shift');
 
     // positioning
     var axLetters = ['x', 'y'];
@@ -74,6 +70,11 @@ function handleSelectionDefaults(selectionIn, selectionOut, fullLayout) {
 
         // Coerce x0, x1, y0, y1
         if(noPath) {
+            if(ax.type === 'category' || ax.type === 'multicategory') {
+                coerce(axLetter + '0shift');
+                coerce(axLetter + '1shift');
+            }
+
             // hack until V3.0 when log has regular range behavior - make it look like other
             // ranges to send to coerce, then put it back after
             // this is all to give reasonable default position behavior on log axes, which is

src/components/shapes/defaults.js

@@ -75,11 +75,6 @@ function handleShapeDefaults(shapeIn, shapeOut, fullLayout) {
         var pos2r;
         var r2pos;
 
-        coerce('x0shift');
-        coerce('x1shift');
-        coerce('y0shift');
-        coerce('y1shift');
-
         // xref, yref
         var axRef = Axes.coerceRef(shapeIn, shapeOut, gdMock, axLetter, undefined,
             'paper');
@@ -90,6 +85,10 @@ function handleShapeDefaults(shapeIn, shapeOut, fullLayout) {
             ax._shapeIndices.push(shapeOut._index);
             r2pos = helpers.rangeToShapePosition(ax);
             pos2r = helpers.shapePositionToRange(ax);
+            if(ax.type === 'category' || ax.type === 'multicategory') {
+                coerce(axLetter + '0shift');
+                coerce(axLetter + '1shift');
+            }
         } else {
             pos2r = r2pos = Lib.identity;
         }