Skip to content

fix(form-field): native select options blending in with dropdown background on a dark theme - #13201

Merged
josephperrott merged 1 commit into
angular:masterfrom
crisbeto:form-field-native-select-dark-theme
Oct 19, 2018
Merged

fix(form-field): native select options blending in with dropdown background on a dark theme#13201
josephperrott merged 1 commit into
angular:masterfrom
crisbeto:form-field-native-select-dark-theme

Conversation

@crisbeto

Copy link
Copy Markdown
Member

When we have a dark theme, we set the color of select to white which ends up propagating to the option elements, causing them to blend in with the background which is always white. These changes set the options explicitly to a dark color for dark themes.

For reference:
angular_material_-_google_chrome_2018-09-19_22-24-13

…ground on a dark theme
When we have a dark theme, we set the color of `select` to white which ends up propagating to the `option` elements, causing them to blend in with the background which is always white. These changes set the options explicitly to a dark color for dark themes.
@crisbetocrisbeto added P2 The issue is important to a large percentage of users, with a workaround target: patch This PR is targeted for the next patch release labels Sep 19, 2018
@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Sep 19, 2018
@crisbeto

Copy link
Copy Markdown
MemberAuthor

Marking as P2 since the issue makes select very hard to use with a dark theme and the fix should be relatively easy to get in.

@mmalerbammalerba added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Oct 18, 2018
@josephperrott
josephperrott merged commit 3f1e960 into angular:masterOct 19, 2018
josephperrott pushed a commit that referenced this pull request Oct 19, 2018
josephperrott pushed a commit that referenced this pull request Oct 19, 2018
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 9, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakercla: yesPR author has agreed to Google's Contributor License AgreementP2The issue is important to a large percentage of users, with a workaroundtarget: patchThis PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@crisbeto@mmalerba@googlebot@josephperrott