Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

fix(select): md-checkbox inside md-list-item using incorrect text color. #7893

Closed

Conversation

DerekLouie
Copy link
Contributor

Hello,

@ThomasBurleson @devversion @EladBezalel.

I used an attribute selector which is not picked up by theming service which made some text miss-colored. Here is the fix + screenshot.

Fixes #7885.

Please Review.

fixedselectcheckboxcss

@DerekLouie DerekLouie self-assigned this Apr 5, 2016
@DerekLouie DerekLouie added the needs: review This PR is waiting on review from the team label Apr 5, 2016
@DerekLouie DerekLouie force-pushed the fixingAttributeSelectorInSelect branch from 3df57b1 to 7bdeba3 Compare April 5, 2016 17:33
@EladBezalel EladBezalel assigned EladBezalel and unassigned DerekLouie Apr 5, 2016
@devversion
Copy link
Member

LGTM

@EladBezalel EladBezalel added needs: manual testing This issue or PR needs to have some manual testing and verification done and removed needs: review This PR is waiting on review from the team labels Apr 5, 2016
@EladBezalel EladBezalel assigned devversion and unassigned EladBezalel Apr 5, 2016
@EladBezalel
Copy link
Member

LGTM @devversion please test manually

@devversion
Copy link
Member

@EladBezalel Yep, this works as expected and fixes the ripple text color side-effect.

@devversion devversion added pr: merge ready This PR is ready for a caretaker to review and removed needs: manual testing This issue or PR needs to have some manual testing and verification done labels Apr 5, 2016
ilovett pushed a commit to ilovett/material that referenced this pull request Apr 22, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

md-checkbox inside md-list-item have wrong colour text in 1.1.0-rc2
4 participants