Skip to content

Conversation

@aveline
Copy link
Contributor

@aveline aveline commented May 31, 2023

WHY are these changes introduced?

Fixes https://github.com/Shopify/polaris-summer-editions/issues/66

WHAT is this pull request doing?

  • Adds experimental value for shadow-inset-lg and shadow-xl tokens
  • Adds shadow-experimental-card-sm and shadow-experimental-card-sm tokens

🎩 checklist

Copy link
Member

@aaronccasanova aaronccasanova left a comment

Choose a reason for hiding this comment

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

Code updates look good 🚀

Copy link

@bernardojoaogarcia bernardojoaogarcia left a comment

Choose a reason for hiding this comment

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

Looks great, you even caught a typo from my end 🙏!

@aveline aveline merged commit 2404594 into v11.x.x May 31, 2023
@aveline aveline deleted the v11xx/shadow-tokens branch May 31, 2023 20:41
sophschneider pushed a commit that referenced this pull request Jun 1, 2023
### WHY are these changes introduced?

Fixes Shopify/archive-polaris-backlog-2024#66

<!--
  Context about the problem that’s being addressed.
-->

### WHAT is this pull request doing?

- Adds experimental value for `shadow-inset-lg` and `shadow-xl` tokens
- Adds `shadow-experimental-card-sm` and `shadow-experimental-card-sm`
tokens

### 🎩 checklist

- [ ] Tested on
[mobile](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md#cross-browser-testing)
- [ ] Tested on [multiple
browsers](https://help.shopify.com/en/manual/shopify-admin/supported-browsers)
- [ ] Tested for
[accessibility](https://github.com/Shopify/polaris/blob/main/documentation/Accessibility%20testing.md)
- [ ] Updated the component's `README.md` with documentation changes
- [ ] [Tophatted
documentation](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting%20documentation.md)
changes in the style guide
sophschneider pushed a commit that referenced this pull request Jun 1, 2023
### WHY are these changes introduced?

Fixes Shopify/archive-polaris-backlog-2024#66

<!--
  Context about the problem that’s being addressed.
-->

### WHAT is this pull request doing?

- Adds experimental value for `shadow-inset-lg` and `shadow-xl` tokens
- Adds `shadow-experimental-card-sm` and `shadow-experimental-card-sm`
tokens

### 🎩 checklist

- [ ] Tested on
[mobile](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md#cross-browser-testing)
- [ ] Tested on [multiple
browsers](https://help.shopify.com/en/manual/shopify-admin/supported-browsers)
- [ ] Tested for
[accessibility](https://github.com/Shopify/polaris/blob/main/documentation/Accessibility%20testing.md)
- [ ] Updated the component's `README.md` with documentation changes
- [ ] [Tophatted
documentation](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting%20documentation.md)
changes in the style guide
aveline added a commit that referenced this pull request Jun 1, 2023
### WHY are these changes introduced?

Fixes https://github.com/Shopify/polaris-summer-editions/issues/66

<!--
  Context about the problem that’s being addressed.
-->

### WHAT is this pull request doing?

- Adds experimental value for `shadow-inset-lg` and `shadow-xl` tokens
- Adds `shadow-experimental-card-sm` and `shadow-experimental-card-sm`
tokens

### 🎩 checklist

- [ ] Tested on
[mobile](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md#cross-browser-testing)
- [ ] Tested on [multiple
browsers](https://help.shopify.com/en/manual/shopify-admin/supported-browsers)
- [ ] Tested for
[accessibility](https://github.com/Shopify/polaris/blob/main/documentation/Accessibility%20testing.md)
- [ ] Updated the component's `README.md` with documentation changes
- [ ] [Tophatted
documentation](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting%20documentation.md)
changes in the style guide
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.

3 participants