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 Button prop name type to variant #18693

Closed
georgewrmarshall opened this issue Apr 20, 2023 · 5 comments
Closed

Update Button prop name type to variant #18693

georgewrmarshall opened this issue Apr 20, 2023 · 5 comments
Assignees
Labels
good first issue Good for newcomers team-design-system All issues relating to design system in Extension

Comments

@georgewrmarshall
Copy link
Contributor

georgewrmarshall commented Apr 20, 2023

Description

Currently the prop to change the button style: PRIMARY, SECONDARY, LINK etc for the component-library Button component is called type. It would be more consistent with the rest of our components of this prop was renamed to variant

There should be only a few instances of the Button component from the ui/components/component-library folder so it should be ok to update all in one PR

Screenshot 2023-04-21 at 1 30 26 AM

Technical Details

  • Update const BUTTON_TYPES to BUTTON_VARIANT
  • Update all instances of type prop to variant for the Button component
  • Update all storybook stories and documentation
  • Update all tests
  • Update all instances of Button used in the code base

Acceptance Criteria

  • All instances of the type prop have been updated to variant
  • All storybook stories and docs have been updated accordingly
  • PR contains before/after screenshots of every instance of the Button that has been updated
@georgewrmarshall georgewrmarshall added good first issue Good for newcomers team-design-system All issues relating to design system in Extension labels Apr 20, 2023
@hakeemullahjan
Copy link
Contributor

@georgewrmarshall I would love to take this. Please can you assign this to me?

@georgewrmarshall
Copy link
Contributor Author

Hey @hakeemullahjan, thanks for your interest. Assigned. Looking forward to your PR. Thanks!

@dhruvv173
Copy link
Contributor

Hi @georgewrmarshall @hakeemullahjan, I'd like to assist to this issue

@georgewrmarshall
Copy link
Contributor Author

Fixed by #18774

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers team-design-system All issues relating to design system in Extension
Projects
None yet
Development

No branches or pull requests

4 participants
@georgewrmarshall @hakeemullahjan @dhruvv173 and others