-
-
Notifications
You must be signed in to change notification settings - Fork 31.7k
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
[TrapFocus] Make an unstable version public #21201
Conversation
Details of bundle changes.Comparing: f62b4b4...e008e93 Details of page changes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that we would need to move the component to its own folder so it can be imported directly like any other
import Unstable_TrapFocus from '@material-ui/core/Unstable_TrapFocus';
This should, in turn, enable API generation, TypeScript to prop-types generation, etc.
Everything works except multiline comment, that for some reason getting more spaces on each run of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, it's a good step forward for https://trello.com/c/SRvmJzrZ/1342-make-focustrap-public.
FYI, this change seems to have broken the latest three alpha builds (5-7) of I had the same experience as this issue in the pickers repo. Oh, the dangers of running with alpha! |
- fix Material UI TrapFocus issue, see: mui/material-ui#21201
<TrapFocus />
needs for pickers migration toPopper
. This PR makes it public and adds typescript defintions.