Skip to content

[docs] Live demos#34454

Merged
michaldudak merged 57 commits intomui:masterfrom
bharatkashyap:live-demos
Oct 17, 2022
Merged

[docs] Live demos#34454
michaldudak merged 57 commits intomui:masterfrom
bharatkashyap:live-demos

Conversation

@bharatkashyap
Copy link
Copy Markdown
Collaborator

@bharatkashyap bharatkashyap commented Sep 24, 2022

@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Oct 17, 2022
Copy link
Copy Markdown
Member

@siriwatknp siriwatknp left a comment

Choose a reason for hiding this comment

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

🤩 Awesome @bharatkashyap. I pushed some fix for Joy UI outlined demos, the rest looks good!

@michaldudak michaldudak merged commit b6e6b23 into mui:master Oct 17, 2022
@michaldudak
Copy link
Copy Markdown
Member

@bharatkashyap Now that this is merged, is #32107 still relevant, or should it be closed?

@bharatkashyap
Copy link
Copy Markdown
Collaborator Author

@bharatkashyap Now that this is merged, is #32107 still relevant, or should it be closed?

Should be good to close this now!

@oliviertassinari
Copy link
Copy Markdown
Member

@nihgwu Thanks for all the help on this one! ❤️

@joserodolfofreitas
Copy link
Copy Markdown
Member

Editing live feels great!
Well done!

@michaldudak
Copy link
Copy Markdown
Member

michaldudak commented Oct 17, 2022

An issue appeared that caused the e2e tests to fail on master (I'm not sure why they passed here): The Material icon browser (see https://deploy-preview-34454--material-ui.netlify.app/material-ui/material-icons/?selected=AcUnit) appears and disappeared a couple of times after loading.
@bharatkashyap do you have any idea what could cause such behavior or should I investigate it in more depth?
Is there a way to disable live edit functionality per demo?

@bharatkashyap
Copy link
Copy Markdown
Collaborator Author

An issue appeared that caused the e2e tests to fail on master (I'm not sure why they passed here): The Material icon browser (see https://deploy-preview-34454--material-ui.netlify.app/material-ui/material-icons/?selected=AcUnit) appears and disappeared a couple of times after loading. @bharatkashyap do you have any idea what could cause such behavior or should I investigate it in more depth? Is there a way to disable live edit functionality per demo?

I'm not exactly certain why this appears @michaldudak - I added a functionality to disable the editing and remove lazy loading in specific demos, but the issue seems to remain. I'm also not certain why the test passed here repeatedly.

@michaldudak
Copy link
Copy Markdown
Member

@siriwatknp managed to fix it in #34802, all is well now.

@oliviertassinari
Copy link
Copy Markdown
Member

oliviertassinari commented Oct 23, 2022

oliviertassinari added a commit to oliviertassinari/material-ui that referenced this pull request Oct 23, 2022
oliviertassinari added a commit to oliviertassinari/material-ui that referenced this pull request Oct 23, 2022
oliviertassinari added a commit to oliviertassinari/material-ui that referenced this pull request Oct 30, 2022
oliviertassinari added a commit to oliviertassinari/material-ui that referenced this pull request Oct 30, 2022
oliviertassinari added a commit to oliviertassinari/material-ui that referenced this pull request Nov 6, 2022
daniel-rabe pushed a commit to daniel-rabe/material-ui that referenced this pull request Nov 29, 2022
Co-authored-by: Neo Nie <[email protected]>
Co-authored-by: Olivier Tassinari <[email protected]>
Co-authored-by: Pedro Ferreira <[email protected]>
Co-authored-by: siriwatknp <[email protected]>
feliperli pushed a commit to jesrodri/material-ui that referenced this pull request Dec 6, 2022
Co-authored-by: Neo Nie <[email protected]>
Co-authored-by: Olivier Tassinari <[email protected]>
Co-authored-by: Pedro Ferreira <[email protected]>
Co-authored-by: siriwatknp <[email protected]>
feliperli pushed a commit to jesrodri/material-ui that referenced this pull request Dec 6, 2022
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants