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.

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

[useMediaQuery] Ensure no tearing in React 18 #30655

Merged
merged 1 commit into from
Jan 26, 2022

Conversation

eps1lon
Copy link
Member

@eps1lon eps1lon commented Jan 16, 2022

Restores #28491 which was reverted in #29870 by using a workaround described in webpack/webpack#14814 (comment).

Failed checks are from the React 18 run and just git diff --exit-code which is expected.

  • React 17 pipeline
  • React 18 pipeline
  • verify locally if a build from this PR works locally in a Create React App but not a build using @mui/material@5.2.0 (which is not using the workaround linked above).

@eps1lon eps1lon added new feature New feature or request hook: useMediaQuery labels Jan 16, 2022
@eps1lon eps1lon changed the title [website] Migrate Twitter from [useMediaQuery] Ensure no tearing in React 18 Jan 16, 2022
@mui-bot
Copy link

mui-bot commented Jan 16, 2022

Details of bundle changes

@material-ui/core: parsed: +0.10% , gzip: +0.13%
@material-ui/lab: parsed: +0.13% , gzip: +0.14%

Generated by 🚫 dangerJS against 07ef237

@eps1lon eps1lon marked this pull request as ready for review January 16, 2022 13:26
@eps1lon eps1lon requested a review from a team January 17, 2022 20:26
Copy link
Member

@michaldudak michaldudak left a comment

Choose a reason for hiding this comment

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

Thanks for working on this! Could you please rebase on the latest master and solve the build errors?
@mnajdova, @siriwatknp you may want to take a look at it as well.

@hbjORbj hbjORbj force-pushed the feat/useMediaQuery/uSES branch from 1467ddf to 07ef237 Compare January 25, 2022 10:15
@eps1lon
Copy link
Member Author

eps1lon commented Jan 25, 2022

Could you please rebase on the latest master and solve the build errors?

Benny rebased now so this should be fine.

For build errors, please see the PR description: They are expected for the React 18 pipeline where failures only related to git-diff. We always have a git-diff change in the React 18 pipeline since we install React 18 via package.json and yarn.lock.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants