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

fix(Button): Deprecate icon, size small properties #362

Merged
merged 4 commits into from
Jul 30, 2020

Conversation

haworku
Copy link
Contributor

@haworku haworku commented Jul 29, 2020

Summary

Button small and icon properties are not usable; the classes they apply do not exist. These properties are now deprecated and will be removed in a future major release.

Background: usa-button--small and usa-button--icon are holdover from a specific Truss project that used custom css.

Related Issues or PRs

closes #282

Copy link
Contributor

@gidjin gidjin left a comment

Choose a reason for hiding this comment

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

Changes look good to me

@haworku haworku added the type: bug Something isn't working like it's supposed to label Jul 29, 2020
Copy link
Contributor

@ahobson ahobson left a comment

Choose a reason for hiding this comment

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

👍

@haworku haworku merged commit 23a2e4b into main Jul 30, 2020
@haworku haworku deleted the hw-bug-small-button-28 branch July 30, 2020 14:42
@haworku haworku mentioned this pull request Aug 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working like it's supposed to
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug] "Small" Button variant results in same styles as default
4 participants