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

If I pass tooltipId to a Tooltip, I see an error in the console.log from the FloatingMenu component #7596

Closed
2 tasks done
scottdickerson opened this issue Jan 19, 2021 · 1 comment · Fixed by #8051
Closed
2 tasks done
Assignees

Comments

@scottdickerson
Copy link
Contributor

What package(s) are you using?

  • carbon-components
  • carbon-components-react

Detailed description

Describe in detail the issue you're having.

Is this issue related to a specific component?
Warning: React does not recognize the tooltipId prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase tooltipid instead. If you accidentally passed it from a parent component, remove it from the DOM element.
in div (created by FloatingMenu)
in div (created by FloatingMenu)
in FloatingMenu (created by Tooltip)
in Tooltip (created by ForwardRef(Tooltip))

What did you expect to happen? What happened instead? What would you like to
see changed?

What browser are you working in?
Chrome

What version of the Carbon Design System are you using?
10.26 and 7.26
What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

Steps to reproduce the issue

To see the console error, go to the CodeSandbox below and expand the Tooltip. You'll see the error get logged

Please create a reduced test case in CodeSandbox

https://codesandbox.io/s/carbon-components-react-forked-gzw8j?file=/src/index.js

@dakahn
Copy link
Contributor

dakahn commented Jan 20, 2021

Thanks for reporting. Moved to backlog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants