-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
md-select placeholder blends with other text. #2465
Comments
@davidwilcox Should be fixed with this PR (#2309). |
Select with placeholder has the same height as a text field (Material guidelines): Material text field specs: |
Tracking via #2124 |
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. |
Bug, feature request, or proposal:
md-select placeholder text is placed under text that occurs immediately prior.
What is the expected behavior?
Padding is built into the md-select so that the placeholder text is not placed over or under any text that comes prior.
What is the current behavior?
The placeholder attribute goes over any text that is prior to it.
What are the steps to reproduce?
Use an md-select element with a placeholder attribute.
http://plnkr.co/edit/R906XtG5GXTA4SNtQUZy?p=preview
Select "Salt Lake City" from the input element.
What is the use-case or motivation for changing an existing behavior?
It looks kind of bad the way it is where you have two things of text on top of each other. It doesn't make sense from a developer's standpoint for this to happen. It's onerous to build into each md-select padding necessary in order to make it look good.
Which versions of Angular, Material, OS, browsers are affected?
@angular/core@^2.4.1
@angular/material@^2.0.0-beta.1
Ubuntu 16.04
Google Chrome browser.
Is there anything else we should know?
Nope! Great job! Thanks for your help!
The text was updated successfully, but these errors were encountered: