jesperct commented on code in PR #41749:
URL: https://github.com/apache/superset/pull/41749#discussion_r3640801445
##########
superset-frontend/plugins/plugin-chart-echarts/src/utils/series.ts:
##########
@@ -365,9 +317,8 @@ export function getLegendLayoutResult({
theme,
type,
}: {
- // Raw chart dimensions. Use availableWidth/availableHeight when other chart
- // UI elements reserve legend space before ECharts lays out the legend.
- availableHeight?: number;
+ // Raw chart dimensions. Use availableWidth when other chart UI elements
+ // reserve legend space before ECharts lays out the legend.
availableWidth?: number;
chartHeight: number;
Review Comment:
chartHeight is still used here. Line 356 reads it as the fallback for
availableHeight (availableHeight ?? chartHeight) when a caller does not pass a
separate height, so dropping it from the signature would break that path.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
