Skip to content

Commit

Permalink
fix: Also constrain aspect ratio
Browse files Browse the repository at this point in the history
  • Loading branch information
bprusinowski committed Jul 3, 2024
1 parent 277e82a commit d4ef8b1
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions app/charts/shared/containers.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -24,16 +24,15 @@ export const useStyles = makeStyles<
width: "100%",
overflow: "hidden",
flexGrow: 1,

},
constrainMinHeight: {
// TODO The aspect ratio is currently set for the whole chart instead of
// affecting only the plot area. Only the plot area should be affected
// otherwise long y-axis ticks squash vertically a chart.
// To remedy, the aspectRatio should be provided via context so that
// the chart state function can get it and apply it to the plot area.
// The ReactGridChartPreview component should also disable this behavior.
aspectRatio: "5 / 2",
},
constrainMinHeight: {
minHeight: 300,
},

Expand Down

0 comments on commit d4ef8b1

Please sign in to comment.