Skip to content

[TrapFocus] Add documentation#22062

Merged
oliviertassinari merged 6 commits intomui:nextfrom
oliviertassinari:trap-focus-docs
Aug 8, 2020
Merged

[TrapFocus] Add documentation#22062
oliviertassinari merged 6 commits intomui:nextfrom
oliviertassinari:trap-focus-docs

Conversation

@oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Aug 3, 2020

@oliviertassinari oliviertassinari added docs Improvements or additions to the documentation. scope: focus trap Changes related to the focus trap. labels Aug 3, 2020
@mui-pr-bot
Copy link

mui-pr-bot commented Aug 3, 2020

Details of bundle changes

Generated by 🚫 dangerJS against 646f0a5

Co-authored-by: Matt <github@nospam.33m.co>
eps1lon
eps1lon previously requested changes Aug 6, 2020
Copy link
Member

@eps1lon eps1lon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Build is failing.

@oliviertassinari oliviertassinari dismissed stale reviews from mbrookes and eps1lon August 6, 2020 11:57

Updated

eps1lon
eps1lon previously requested changes Aug 6, 2020
Copy link
Member

@eps1lon eps1lon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The page should have a big warning that this component is experimental.

oliviertassinari and others added 3 commits August 6, 2020 19:54
Co-authored-by: Matt <github@nospam.33m.co>
eps1lon
eps1lon previously requested changes Aug 7, 2020
@oliviertassinari oliviertassinari dismissed eps1lon’s stale review August 7, 2020 15:39

thanks for the review, I have updated the changes to take them into account

Copy link
Member

@eps1lon eps1lon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Already found a bug with existing components 😄

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

Labels

docs Improvements or additions to the documentation. scope: focus trap Changes related to the focus trap.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants