-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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(shape): Rename surface term with component. #3761
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more readme change, then LGTM
packages/mdc-shape/README.md
Outdated
`$mdc-shape-medium-surface-radius` | Rounded shape radius size for medium surface components. Default value `8px`. | ||
`$mdc-shape-large-surface-radius` | Rounded shape radius size for large surface components. Default value `0`. | ||
`$mdc-shape-small-component-radius` | Rounded shape radius size for small components. Default value `4px`. | ||
`$mdc-shape-medium-component-radius` | Rounded shape radius size for medium components. Default value `8px`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While we're here: this says 8px is the default, but it's 4px in the code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. Updated the docs.
All 557 screenshot tests passed for commit f228de5 vs. |
All 557 screenshot tests passed for commit 1a9b8d7 vs. |
All 557 screenshot tests passed for commit eba1a35 vs. |
All 557 screenshot tests passed for commit f26a7f8 vs. |
BREAKING CHANGE: Renamed shape global variables from
$mdc-shape-*-surface-radius
to$mdc-shape-*-component-radius