Calculate the maximum height instead of adjusting initial column heights

chartjs · Oct 20, 2018 · 7a58896 · 7a58896
1 parent e8f9c50
commit 7a58896
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/plugins/plugin.legend.js b/src/plugins/plugin.legend.js
@@ -272,21 +272,22 @@ var Legend = Element.extend({
 				var vPadding = labelOpts.padding;
 				var columnWidths = me.columnWidths = [];
 				var totalWidth = labelOpts.padding;
+				var maxHeight = minSize.height - vPadding;
 				var currentColWidth = 0;
-				var currentColHeight = vPadding;
+				var currentColHeight = 0;
 				var itemHeight = fontSize + vPadding;
 
 				helpers.each(me.legendItems, function(legendItem, i) {
 					var boxWidth = getBoxWidth(labelOpts, fontSize);
 					var itemWidth = boxWidth + (fontSize / 2) + ctx.measureText(legendItem.text).width;
 
 					// If too tall, go to new column
-					if (currentColHeight + itemHeight > minSize.height) {
+					if (currentColHeight + itemHeight > maxHeight) {
 						totalWidth += currentColWidth + labelOpts.padding;
 						columnWidths.push(currentColWidth); // previous column width
 
 						currentColWidth = 0;
-						currentColHeight = vPadding;
+						currentColHeight = 0;
 					}
 
 					// Get max width