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

Author: hannahker

src/components/legend/attributes.js

@@ -159,16 +159,15 @@ module.exports = {
     },
     x: {
         valType: 'number',
-        min: -2,
-        max: 3,
         editType: 'legend',
         description: [
             'Sets the x position with respect to `xref` (in normalized coordinates) of the legend.',
             'When `xref` is *paper*, defaults to *1.02* for vertical legends and',
             'defaults to *0* for horizontal legends.',
             'When `xref` is *container*, defaults to *1* for vertical legends and',
             'defaults to *0* for horizontal legends.',
-            '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: {
@@ -198,16 +197,15 @@ module.exports = {
     },
     y: {
         valType: 'number',
-        min: -2,
-        max: 3,
         editType: 'legend',
         description: [
             'Sets the y position with respect to `yref` (in normalized coordinates) of the legend.',
             'When `yref` is *paper*, defaults to *1* for vertical legends,',
             'defaults to *-0.1* for horizontal legends on graphs w/o range sliders and',
             'defaults to *1.1* for horizontal legends on graph with one or multiple range sliders.',
             'When `yref` is *container*, defaults to *1*.',
-            '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: {

src/components/legend/defaults.js

@@ -147,6 +147,26 @@ function groupDefaults(legendId, layoutIn, layoutOut, fullData) {
         }
     }
 
+    Lib.coerce(containerIn, containerOut, {
+        x: {
+            valType: 'number',
+            editType: 'legend',
+            min: isPaperX ? -2 : 0,
+            max: isPaperX ? 3 : 1,
+            dflt: defaultX,
+        }
+    }, 'x');
+
+    Lib.coerce(containerIn, containerOut, {
+        y: {
+            valType: 'number',
+            editType: 'legend',
+            min: isPaperY ? -2 : 0,
+            max: isPaperY ? 3 : 1,
+            dflt: defaultY,
+        }
+    }, 'y');
+
     coerce('traceorder', defaultOrder);
     if(helpers.isGrouped(layoutOut.legend)) coerce('tracegroupgap');