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

[test] Fix browser tests #29505

Merged
merged 2 commits into from
Nov 5, 2021
Merged

[test] Fix browser tests #29505

merged 2 commits into from
Nov 5, 2021

Conversation

eps1lon
Copy link
Member

@eps1lon eps1lon commented Nov 4, 2021

Fixes https://app.circleci.com/jobs/github/mui-org/material-ui/312033
Browser run: https://app.circleci.com/pipelines/github/mui-org/material-ui/54574/workflows/8d02f99c-3236-43c6-94d7-63eb76d1ed72

  1. Use longhand properties instead of shorthand properties such as border-radius
  2. Adjust font-weight depending on browser
  3. adjust font-family computation depending on browser

@eps1lon eps1lon added the test label Nov 4, 2021
@mui-pr-bot
Copy link

mui-pr-bot commented Nov 4, 2021

No bundle size changes

Generated by 🚫 dangerJS against 37b57aa

@eps1lon eps1lon requested a review from siriwatknp November 4, 2021 11:59
@eps1lon eps1lon marked this pull request as ready for review November 4, 2021 11:59
Copy link
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.

👍 noted, thanks.

@eps1lon eps1lon merged commit c51c79b into mui:master Nov 5, 2021
@eps1lon eps1lon deleted the test/browser-tests branch November 5, 2021 10:19
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