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

Update add action pattern guidance #11906

Merged
merged 1 commit into from
Apr 18, 2024
Merged

Update add action pattern guidance #11906

merged 1 commit into from
Apr 18, 2024

Conversation

sarahill
Copy link
Contributor

Updated add action guidance to make icon usage clearer.

before after
Screenshot 2024-04-18 at 9 07 29 AM Screenshot 2024-04-18 at 9 07 00 AM

@sarahill sarahill self-assigned this Apr 18, 2024
@sarahill sarahill added the #gsd:38421 Polaris Common Action Patterns label Apr 18, 2024
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.

@sarahill sarahill merged commit 5b9f4b8 into main Apr 18, 2024
11 checks passed
@sarahill sarahill deleted the add-action-pattern-update branch April 18, 2024 17:23
kyledurand pushed a commit that referenced this pull request Apr 19, 2024
This PR was opened by the [Changesets
release](https:/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @shopify/[email protected]

### Patch Changes

- [#11915](#11915)
[`828995cc4`](828995c)
Thanks [@kyledurand](https:/kyledurand)! - Fixed Picker
empty state


- [#11918](#11918)
[`fd37c1896`](fd37c18)
Thanks [@kyledurand](https:/kyledurand)! - Reverted selected
icon placement on Option and TextOption

## [email protected]

### Patch Changes

- [#11906](#11906)
[`5b9f4b898`](5b9f4b8)
Thanks [@sarahill](https:/sarahill)! - Updated common action
pattern add guiance


- [#11909](#11909)
[`c4f766f1d`](c4f766f)
Thanks [@sarahill](https:/sarahill)! - Fixed missing image
for card layout pattern documentation

- Updated dependencies
\[[`828995cc4`](828995c),
[`fd37c1896`](fd37c18)]:
    -   @shopify/[email protected]

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
AnnaCheba pushed a commit to AnnaCheba/polaris that referenced this pull request Apr 22, 2024
Updated add action guidance to make icon usage clearer.

before | after
--|--
![Screenshot 2024-04-18 at 9 07
29 AM](https:/Shopify/polaris/assets/8629173/152659ba-8680-4bd4-87bf-e926ceadb140)
| ![Screenshot 2024-04-18 at 9 07
00 AM](https:/Shopify/polaris/assets/8629173/9382be14-3dcc-4234-8b98-c8a2e2e0564b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#gsd:38421 Polaris Common Action Patterns
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants