-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
refactor(material/button): handle disabled states through tokens #28005
Conversation
@@ -89,15 +88,3 @@ | |||
$query: mdc-helpers.$mdc-base-styles-query); | |||
} | |||
} | |||
|
|||
// Changes a button token set to exclude the ripple styles. | |||
@function mat-private-button-remove-ripple($tokens) { |
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.
Unrelated change, I just noticed that we weren't using this.
MDC applies disabled styles through the `:disabled` selector which forced us to work around it by using a selector like `.mat-mdc-button[disabled][disabled]` and setting both the disabled and enabled button tokens. This is problematic, because it increases the specificity too much and it introduces space for mistakes, because we have to duplicate the tokens values. These changes resolve the issue by re-applying the token slots to disabled buttons with the correct selector so that they can reuse the same tokens as the `button` nodes.
0b14e66
to
78694f4
Compare
Merging this manually through the UI. Will send a follow-up to cherry-pick into the RC branch. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
MDC applies disabled styles through the
:disabled
selector which forced us to work around it by using a selector like.mat-mdc-button[disabled][disabled]
and setting both the disabled and enabled button tokens. This is problematic, because it increases the specificity too much and it introduces space for mistakes, because we have to duplicate the tokens values.These changes resolve the issue by re-applying the token slots to disabled buttons with the correct selector so that they can reuse the same tokens as the
button
nodes.