Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[charts] slotProps error #10132

Closed
2 tasks done
jared-craig-cse opened this issue Aug 24, 2023 · 1 comment
Closed
2 tasks done

[charts] slotProps error #10132

jared-craig-cse opened this issue Aug 24, 2023 · 1 comment
Labels
component: charts This is the name of the generic UI component, not the React module! duplicate This issue or pull request already exists

Comments

@jared-craig-cse
Copy link

Duplicates

  • I have searched the existing issues

Latest version

  • I have tested the latest version

Steps to reproduce 🕹

Updated from x-charts alpha 6 to alpha 7.

Current behavior 😯

I get a console error after updating to alpha 7:

React does not recognize the slotProps prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase slotprops instead. If you accidentally passed it from a parent component, remove it from the DOM element.

I am currently not using the slots or slotProps properties.

Expected behavior 🤔

There shouldn't be a console error for the new feature.

Context 🔦

I'm currently displaying a few line charts that were working with no errors on alpha 6. They continue to work but with the mentioned console error.

Your environment 🌎

System:
OS: Windows 10 10.0.19045
Binaries:
Node: 18.14.1 - C:\Program Files\nodejs\node.EXE
Yarn: Not Found
npm: 9.6.5 - C:\Program Files\nodejs\npm.CMD
Browsers:
Chrome: 116.0.5845.98
Edge: Spartan (44.19041.1266.0), Chromium (115.0.1901.203)
npmPackages:
@emotion/react: ^11.11.1 => 11.11.1
@emotion/styled: ^11.11.0 => 11.11.0
@mui/base: 5.0.0-beta.12
@mui/core-downloads-tracker: 5.14.6
@mui/icons-material: ^5.14.3 => 5.14.3
@mui/material: ^5.14.6 => 5.14.6
@mui/private-theming: 5.14.6
@mui/styled-engine: 5.14.6
@mui/system: 5.14.6
@mui/types: 7.2.4
@mui/utils: 5.14.6
@mui/x-charts: ^6.0.0-alpha.7 => 6.0.0-alpha.7
@mui/x-data-grid: 6.11.2
@mui/x-data-grid-pro: ^6.11.2 => 6.11.2
@mui/x-date-pickers: 6.11.1
@mui/x-date-pickers-pro: ^6.11.1 => 6.11.1
@mui/x-license-pro: 6.10.2
@types/react: ^18.0.21 => 18.2.17
react: ^18.2.0 => 18.2.0
react-dom: ^18.2.0 => 18.2.0
typescript: ^4.9.5 => 4.9.5

Order ID or Support key 💳 (optional)

No response

@jared-craig-cse jared-craig-cse added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Aug 24, 2023
@LukasTy
Copy link
Member

LukasTy commented Aug 25, 2023

Thank you for reporting the issue and using the freshly baked charts package! 🙏 💯
This seems to be a duplicate of #10072 and has already been resolved with #10105.
We ought to release a new package today. 😉

@LukasTy LukasTy closed this as not planned Won't fix, can't repro, duplicate, stale Aug 25, 2023
@LukasTy LukasTy added duplicate This issue or pull request already exists component: charts This is the name of the generic UI component, not the React module! and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: charts This is the name of the generic UI component, not the React module! duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants