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

[withMobileDialog] Remove #23020

Closed
wants to merge 2 commits into from
Closed

[withMobileDialog] Remove #23020

wants to merge 2 commits into from

Conversation

fabiothiroki
Copy link

@fabiothiroki fabiothiroki commented Oct 12, 2020 •
edited
Loading

Add a test case when no breakpoint nor options is passed as argument

@fabiothiroki fabiothiroki changed the title Add test for default case Add test for withMobileDialog Oct 12, 2020
@mui-pr-bot
Copy link

No bundle size changes comparing dcbcb39...5d409f7

Generated by 🚫 dangerJS against 5d409f7

@oliviertassinari
Copy link
Member

@fabiothiroki The module is going out, see #14992

@oliviertassinari oliviertassinari changed the title Add test for withMobileDialog [withMobileDialog] Remove Oct 12, 2020
@fabiothiroki
Copy link
Author

@fabiothiroki The module is going out, see #14992

Thanks, closing this PR now

@fabiothiroki fabiothiroki deleted the add-test-withMobileDialog branch October 12, 2020 18:28
@oliviertassinari
Copy link
Member

@fabiothiroki If you want to help us remove the component from the source, it would be welcome

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
breaking change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fabiothiroki @mui-pr-bot @oliviertassinari