mdc-button--theme-dark with :disabled, doesn't properly set background-color or color

[lynnmercier]

Check out the latest code and run npm run dev, navigate to http://localhost:8080/button.html (The latest version of the demo page is not available on https://material-components-web.appspot.com/button.html yet)

The mdc-theme--dark section shows a disabled button with grey text

However, the mdc-button--theme-dark section shows a disabled button with black text.

The second section, mdc-button--theme-dark, has the bug. The disabled button should have grey text.

The fix is to correctly use the @include mdc-theme-dark(".mdc-button") within the :disabled context of mdc-button.scss. It needs to be updated to

@include mdc-theme-dark(".mdc-button", true /* compound */)

mdc-theme-dark mixin requires $compound to be set to true if the

current selector is a compound selector with the base selector

...and :disabled is a compound selector with .mdc-button.

[bhongy]

As of the current HEAD on master f0b607, this seems to be fixed. Screenshot below.

@lynnjepsen is this still an issue?

Note: anchor tag (Link) does not receive disabled style but that seems to be a separate issue.

[acdvorak]

Fixed by @yeelan0319.