Skip to content

Conversation

@sophschneider
Copy link
Contributor

WHY are these changes introduced?

Fixes #9965

WHAT is this pull request doing?

Consolidate se23 logic and styles for ShadowBevel Component

How to 🎩

  • Compare production and this PR's chromatic storybook to make sure styles are the same
  • polarisSummerEditions2023ShadowBevelOptOut toggle shouldn't do anything now

Production
This PR

@sophschneider sophschneider marked this pull request as ready for review August 21, 2023 19:41
Copy link
Contributor

@laurkim laurkim left a comment

Choose a reason for hiding this comment

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

💯

@sophschneider sophschneider merged commit 4de0f6a into next Aug 22, 2023
@sophschneider sophschneider deleted the sophie/shadow-bevel-clean-up branch August 22, 2023 14:23
sophschneider added a commit that referenced this pull request Sep 19, 2023
### WHY are these changes introduced?

Fixes #9965

### WHAT is this pull request doing?

Consolidate se23 logic and styles for `ShadowBevel` Component

### How to 🎩
* Compare production and this PR's chromatic storybook to make sure
styles are the same
* `polarisSummerEditions2023ShadowBevelOptOut` toggle shouldn't do
anything now


[Production](https://storybook.polaris.shopify.com/?path=/story/all-components-shadowbevel--default&globals=polarisSummerEditions2023:true)
[This PR]()
AnnaCheba pushed a commit to AnnaCheba/polaris that referenced this pull request Apr 22, 2024
### WHY are these changes introduced?

Fixes Shopify#9965

### WHAT is this pull request doing?

Consolidate se23 logic and styles for `ShadowBevel` Component

### How to 🎩
* Compare production and this PR's chromatic storybook to make sure
styles are the same
* `polarisSummerEditions2023ShadowBevelOptOut` toggle shouldn't do
anything now


[Production](https://storybook.polaris.shopify.com/?path=/story/all-components-shadowbevel--default&globals=polarisSummerEditions2023:true)
[This PR]()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants