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.

Sign up for GitHub

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

Jump to bottom

Speed Dial throws in MUI demos #17293

Closed
2 tasks done
lsnch opened this issue Sep 3, 2019 · 0 comments · Fixed by #17301
Closed
2 tasks done

Speed Dial throws in MUI demos #17293

lsnch opened this issue Sep 3, 2019 · 0 comments · Fixed by #17301
Assignees
@oliviertassinari
Labels
component: speed dial This is the name of the generic UI component, not the React module!

Comments

@lsnch
Copy link
Contributor

lsnch commented Sep 3, 2019

  • The issue is present in the latest release.
  • I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

Focus dial, press ESC. This error is then thrown in console:
Uncaught TypeError: Cannot read property 'focus' of undefined

Expected Behavior 🤔

There should be no error and ESC should close the dial.

Steps to Reproduce 🕹

https://material-ui.com/components/speed-dial/

Tech Version
Material-UI
React
Browser Chrome Version 76.0.3809.100
TypeScript
etc.
@mbrookes mbrookes added the component: speed dial This is the name of the generic UI component, not the React module! label Sep 3, 2019
@oliviertassinari oliviertassinari self-assigned this Sep 9, 2019
@hashwin hashwin mentioned this issue Sep 10, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oliviertassinari oliviertassinari

Labels
component: speed dial This is the name of the generic UI component, not the React module!
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[SpeedDial] Rework part of the logic hashwin/material-ui
3 participants
@mbrookes @oliviertassinari @lsnch