search

mui / mui-x

MUI X: Build complex and data-rich applications using a growing list of advanced React components, like the Data Grid, Date and Time Pickers, Charts, and more!
https://mui.com/x/
3.88k stars 1.17k forks source link

[charts][BarChart] axis categoryGapRatio and barGapRatio gives TS errors #13154

Closed apdrsn closed 1 month ago

apdrsn commented 1 month ago

Steps to reproduce

Link to live example: https://codesandbox.io/p/sandbox/xenodochial-star-qjthpl (fork from this example https://mui.com/x/react-charts/bars/#bar-direction)

Steps:

  1. I changed the forked package.json to have "latest" instead of "next" for the @mui/material package
  2. Throws a ts error image

Current behavior

No response

Expected behavior

No response

Context

No response

Your environment

npx @mui/envinfo ``` Don't forget to mention which browser you used. Output from `npx @mui/envinfo` goes here. ```

Search keywords: categoryGapRatio barGapRatio Order ID: 90196

apdrsn commented 1 month ago

Hi @michelengelen. My colleague just found this issue, which seems to be related. https://github.com/mui/mui-x/issues/10334

michelengelen commented 1 month ago

Hi @michelengelen. My colleague just found this issue, which seems to be related. #10334

Yes and the workaround there is also what I found. For this to work properly without casting we would probably need to do some typing magic.

@alexfauquette lets track #10334 for the effort and close this one.

github-actions[bot] commented 1 month ago

:warning: This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue. Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

@apdrsn: How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey.