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

Some primary colours are not set with variables in CSS #3441

Closed
Bloemendaal opened this issue Aug 28, 2018 · 2 comments
Closed

Some primary colours are not set with variables in CSS #3441

Bloemendaal opened this issue Aug 28, 2018 · 2 comments

Comments

@Bloemendaal
Copy link

Bugs

Follow the template below and include a codepen to ensure the quickest and most accurate response to your issue.

What MDC Web Version are you using?

0.39.0

What browser(s) is this bug affecting?

Chrome 68 (latest)

What OS are you using?

Windows 10 Home

In short

Please add variables to the following classes to have them match the primary colour in my project.
Thanks in advance, I love this project, keep up the great work :)

What is the expected behavior?

.mdc-text-field--focused:not(.mdc-text-field--disabled) .mdc-floating-label { color: rgba(98,0,238,0.87); color: var(--mdc-theme-primary, #6200ee); }

.mdc-drawer .mdc-list-item--activated { color: rgba(98,0,238,0.87); color: var(--mdc-theme-primary, #6200ee); }

.mdc-drawer .mdc-list-item--activated .mdc-list-item__graphic { color: #6200ee; color: var(--mdc-theme-primary, #6200ee); }

What is the actual behavior?

.mdc-text-field--focused:not(.mdc-text-field--disabled) .mdc-floating-label { color: rgba(98,0,238,0.87); }

.mdc-drawer .mdc-list-item--activated { color: rgba(98,0,238,0.87); }

.mdc-drawer .mdc-list-item--activated .mdc-list-item__graphic { color: #6200ee; }

@williamernest
Copy link
Contributor

@Bloemendaal Thanks for filing this issue. This is already being tracked in #3066. In order to closely match spec, we ended up breaking css variable support for some of our components and are investigating how we can resolve the issue.

@williamernest
Copy link
Contributor

Closing this issue since it's already being tracked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants