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
Upgrade styled-components to v5 #3397
Conversation
🦋 Changeset detectedLatest commit: c0dac98 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@media screen and (min-width:544px) { | ||
|
||
} | ||
|
||
@media screen and (min-width:768px) { | ||
|
||
} | ||
|
||
@media screen and (min-width:1012px) { | ||
|
||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure what's causing these to get output
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like this is a know issue both for us and for the community
That said since we are moving away from jest snapshots (ADR reference), if this is only snapshot issue (seems like it) we might maybe want to start deleting these snapshots? At least we can start with Box maybe? And if needed we can create a test in Playwright. What do you think?
size-limit report 📦
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me 🥳 What would the best next steps for this be? Would it be going through and doing a visual inspection to make sure nothing was impacted?
Co-authored-by: Josh Black <joshblack@github.com>
@joshblack - I already went through a bunch of stuff in Storybook, but maybe we could visually review in dotcom |
…eact into mp/styled-components-v5-upgrade
@broccolinisoup - I think we can just delete that snapshot |
…eact into mp/styled-components-v5-upgrade
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥳
I think so too! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥳
This reverts commit 397938d.
Merge checklist
Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.