onResize and onResizeCapture error in GridActionsCellItem

[karolis-666]

Duplicates

• [X] I have searched the existing issues

Latest version

• [X] I have tested the latest version

Steps to reproduce 🕹

Steps:

1. Import <GridActionsCellItem /> from "@mui/x-data-grid-pro"
2. Use <GridActionsCellItem />

Current behavior 😯

Component <GridActionsCellItem /> raises TS error, that props onResize, onResizeCapture and nonce are not provided. Proving them with values undefined raises erros in browser and our build.

Expected behavior 🤔

No TS error when not using these props

Context 🔦

Seems that this issue was fixed https://github.com/mui/material-ui/issues/35287 only for <Popper />, but not for <GridActionsCellItem /> as we also encountered issues with <Popper /> that are now fixed

Your environment 🌎

"@mui/base": "^5.0.0-alpha.90",
"@mui/icons-material": "^5.8.4",
"@mui/lab": "^5.0.0-alpha.86",
"@mui/material": "^5.11.10",
"@mui/styles": "^5.8.4",
"@mui/system": "^5.8.4",
"@mui/x-data-grid": "^5.12.2",
"@mui/x-data-grid-premium": "^5.17.24",
"@mui/x-data-grid-pro": "^5.17.24",
"@mui/x-date-pickers-pro": "^5.0.0-alpha.7",
"@mui/x-license-pro": "^5.13.1",
"@types/react": "^17.0.47",
"react": "^17.0.2",

Order ID 💳 (optional)

46744

[m4theushw]

I added it to our backlog.

[bagoette]

This same issue appears to be on the components GridPagination and GridToolbarDensitySelector

[jacksonlauder]

This issue also appears with GridToolbarColumnsButton

[cherniavskii]

Can anyone provide a reproduction example for this issue? I tried to reproduce it using react@17 and @types/react@17, but don't see any errors here: https://codesandbox.io/s/nifty-rosalind-xvxju4?file=/index.tsx

[karolis-666]

Codesandbox can't find @mui/x-data-grid-premium package, but here is link anyways: https://codesandbox.io/s/admiring-yalow-76xglk?file=/index.tsx

Issue can be seen using premium or pro package

[cherniavskii]

@karolis-666 I've forked your sandbox, installed @mui/x-data-grid-premium, but there are no TS errors: https://codesandbox.io/s/confident-hooks-50b6jh?file=/index.tsx

Same in my local project with the same setup:

We need a reproduction to take an action on it. Can you setup a repo with a reproduction example maybe?

[karolis-666]

I setup repo, but while setting it up encountered other TS errors with same component in question. Link: https://github.com/karolis-666/mui-1

[cherniavskii]

Thanks @karolis-666 I forked your repo and removed some unnecessary dependencies. I can definitely reproduce the issue:

---

Then I tried upgrading the data grid to v6 and pushed it to a separate branch: https://github.com/cherniavskii/mui-1/tree/v6 And v6 works fine in the same setup:

---

Could you try reproducing it using data grid v6 in https://github.com/cherniavskii/mui-1/tree/v6?

[karolis-666]

In v6 I can't reproduced it so it seems solved. Unfortunately v6 is still in pre-release phase and experienced other issues with useGridApiContext hook so we can't upgrade to that version yet.

[cherniavskii]

Hi @karolis-666 Sorry for the late reply! The v6 stable was released in early March https://github.com/mui/mui-x/releases/tag/v6.0.0 This is now the officially supported major version, where we'll keep rolling out new features, bug fixes, and improvements.

[cherniavskii]

Since the issue is solved in v6, I'm going to close this issue as there's no action to be taken on our end.

[nouman91]

I am using version 5 and I can't upgrade, can we do a bug fix for v5?