fix non-ordinal autoHeight

mbostock · mbostock · commit 42d95b457ae1 · 2021-05-01T19:43:22.000-07:00
diff --git a/src/plot.js b/src/plot.js
@@ -138,7 +138,7 @@ function ScaleFunctions(scales) {
 }
 
 function autoHeight({y, fy, fx}) {
-  const ny = y && y.type === "ordinal" ? y.scale.domain().length : 6;
+  const ny = y && y.type === "ordinal" ? y.scale.domain().length : 17;
   const nfy = fy ? fy.scale.domain().length : 1;
   return !!(y || fy) * Math.max(1, Math.min(60, ny * nfy)) * 20 + !!fx * 30 + 60;
 }

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ function ScaleFunctions(scales) {`
`138`	`138`	`}`
`139`	`139`
`140`	`140`	`function autoHeight({y, fy, fx}) {`
`141`		`- const ny = y && y.type === "ordinal" ? y.scale.domain().length : 6;`
	`141`	`+ const ny = y && y.type === "ordinal" ? y.scale.domain().length : 17;`
`142`	`142`	`const nfy = fy ? fy.scale.domain().length : 1;`
`143`	`143`	`return !!(y \|\| fy) * Math.max(1, Math.min(60, ny * nfy)) * 20 + !!fx * 30 + 60;`
`144`	`144`	`}`