components / legend / add hmaxheightratio, hmaxheight attributes

  - Replace hardcoded horizontal legend max height ratio of 2 with
  user options (default is still 2).

Author: Nementon

src/components/legend/attributes.js

@@ -33,6 +33,24 @@ module.exports = {
         editType: 'legend',
         description: 'Sets the color of the border enclosing the legend.'
     },
+    hmaxheightratio: {
+        valType: 'number',
+        min: 2,
+        dflt: 2,
+        role: 'style',
+        editType: 'legend',
+        description: [
+	    'Sets the max height ratio (layout / ratio) of the visible legend when horizontaly aligned.',
+	    'Default value is 2; the legend will take up to 50% of the layout height before displaying a scrollbar',
+	].join(' ')
+    },
+    hmaxheight: {
+        valType: 'number',
+        min: 0,
+        role: 'style',
+        editType: 'legend',
+        description: 'Sets the max height (in px) of the horizontaly aligned legend.'
+    },
     borderwidth: {
         valType: 'number',
         min: 0,

src/components/legend/defaults.js

@@ -188,6 +188,8 @@ function groupDefaults(legendId, layoutIn, layoutOut, fullData) {
 
     coerce('xanchor', defaultXAnchor);
     coerce('yanchor', defaultYAnchor);
+    coerce('hmaxheightratio');
+    coerce('hmaxheight');
     coerce('valign');
     Lib.noneOrAll(containerIn, containerOut, ['x', 'y']);
 

src/components/legend/draw.js

@@ -769,12 +769,18 @@ function computeLegendDimensions(gd, groups, traces, legendObj) {
     var traceGroupGap = legendObj.tracegroupgap;
     var legendGroupWidths = {};
 
-    // - if below/above plot area, give it the maximum potential margin-push value
+    // - if below/above plot area, give it the [user defined] maximum potential margin-push value
     // - otherwise, extend the height of the plot area
-    legendObj._maxHeight = Math.max(
-        (isBelowPlotArea || isAbovePlotArea) ? fullLayout.height / 2 : gs.h,
-        30
-    );
+    if (isBelowPlotArea || isAbovePlotArea) {
+        if (legendObj.hmaxheight !== undefined) {
+            legendObj._maxHeight = legendObj.hmaxheight;
+        } else { 
+            legendObj._maxHeight = fullLayout.height / legendObj.hmaxheightratio; 
+        }
+    }
+    else { 
+        legendObj._maxHeight = Math.max(gs.h, 30);
+    }
 
     var toggleRectWidth = 0;
     legendObj._width = 0;