-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Create Button
migrations
#10161
Labels
#gsd:36573
Polaris API Alignment
Comments
Closed
This was referenced Aug 28, 2023
aveline
added a commit
that referenced
this issue
Sep 6, 2023
### WHY are these changes introduced? Fixes #10161 ### WHAT is this pull request doing? Migration updates `920` files in `web` Adds comments to `1650` files to be updated manually ### How to 🎩 🖥 [Local development instructions](https://github.com/Shopify/polaris/blob/main/README.md#local-development) 🗒 [General tophatting guidelines](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md) 📄 [Changelog guidelines](https://github.com/Shopify/polaris/blob/main/.github/CONTRIBUTING.md#changelog) <!-- Give as much information as needed to experiment with the component in the playground. --> <details> <summary>Copy-paste this code in <code>playground/Playground.tsx</code>:</summary> ```jsx import React from 'react'; import {Page} from '../src'; export function Playground() { return ( <Page title="Playground"> {/* Add the code you want to test in here */} </Page> ); } ``` </details> ### 🎩 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 --------- Co-authored-by: Lo Kim <26749317+laurkim@users.noreply.github.com> Co-authored-by: Aaron Casanova <32409546+aaronccasanova@users.noreply.github.com>
aveline
added a commit
that referenced
this issue
Sep 6, 2023
sophschneider
pushed a commit
that referenced
this issue
Sep 19, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
primary
prop tovariant="primary"
plain
prop tovariant="plain"
primary
plain
props tovariant="tertiary"
plain
monochrome
tovariant="monochromePlain"
destructive
prop totone="critical"
variant="primary"
outline
destructive
props totone="critical"
outline
propoutline
monochrome
propmonochrome
propbuttonFrom
functionThe text was updated successfully, but these errors were encountered: