Conversation
…ions React Runner implementation - fix error spacing and blurred input text
siriwatknp
left a comment
There was a problem hiding this comment.
🤩 Awesome @bharatkashyap. I pushed some fix for Joy UI outlined demos, the rest looks good!
|
@bharatkashyap Now that this is merged, is #32107 still relevant, or should it be closed? |
Should be good to close this now! |
|
@nihgwu Thanks for all the help on this one! ❤️ |
|
Editing live feels great! |
|
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. |
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. |
|
@siriwatknp managed to fix it in #34802, all is well now. |
This reverts commit b6e6b23.
This reverts commit 19df4df.
This reverts commit 19df4df.
This reverts commit 19df4df.
This reverts commit 19df4df.
This reverts commit 19df4df.
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]>
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]>
This reverts commit b6e6b23.
https://deploy-preview-34454--material-ui.netlify.app/material-ui/react-button/
Continuing the effort started in #24640, #30873 and #32107