Coerce min/max of x and y depending on xref and yref

Author: hannahker

Diff for: src/components/colorbar/attributes.js

@@ -57,15 +57,14 @@ module.exports = overrideAll({
     },
     x: {
         valType: 'number',
-        min: -2,
-        max: 3,
         description: [
             'Sets the x position with respect to `xref` of the color bar (in plot fraction).',
             'When `xref` is *paper*, defaults to 1.02 when `orientation` is *v* and',
             '0.5 when `orientation` is *h*.',
             'When `xref` is *container*, defaults to *1* when `orientation` is *v* and',
             '0.5 when `orientation` is *h*.',
-            'Must be between *0* and *1* if `xref` is *container*.'
+            'Must be between *0* and *1* if `xref` is *container*',
+            'and between *-2* and *3* if `xref` is *paper*.'
         ].join(' ')
     },
     xref: {
@@ -98,15 +97,14 @@ module.exports = overrideAll({
     },
     y: {
         valType: 'number',
-        min: -2,
-        max: 3,
         description: [
             'Sets the y position with respect to `yref` of the color bar (in plot fraction).',
             'When `yref` is *paper*, defaults to 0.5 when `orientation` is *v* and',
             '1.02 when `orientation` is *h*.',
             'When `yref` is *container*, defaults to 0.5 when `orientation` is *v* and',
             '1 when `orientation` is *h*.',
-            'Must be between *0* and *1* if `yref` is *container*.'
+            'Must be between *0* and *1* if `yref` is *container*',
+            'and between *-2* and *3* if `yref` is *paper*.'
         ].join(' ')
     },
     yref: {

Diff for: src/components/colorbar/defaults.js

@@ -58,10 +58,26 @@ module.exports = function colorbarDefaults(containerIn, containerOut, layout) {
         defaultY = isPaperY ? 1.02 : 1;
     }
 
-    coerce('x', defaultX);
+    Lib.coerce(colorbarIn, colorbarOut, {
+        x: {
+            valType: 'number',
+            min: isPaperX ? -2 : 0,
+            max: isPaperX ? 3 : 1,
+            dflt: defaultX,
+        }
+    }, 'x');
+
+    Lib.coerce(colorbarIn, colorbarOut, {
+        y: {
+            valType: 'number',
+            min: isPaperY ? -2 : 0,
+            max: isPaperY ? 3 : 1,
+            dflt: defaultY,
+        }
+    }, 'y');
+
     coerce('xanchor', defaultXAnchor);
     coerce('xpad');
-    coerce('y', defaultY);
     coerce('yanchor', defaultYAnchor);
     coerce('ypad');
     Lib.noneOrAll(colorbarIn, colorbarOut, ['x', 'y']);