-
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
bug(md-input): MdInput with type date cannot handle placeholders #845
Comments
The date input will get a picker. For now I'd suggest styling the label as if it was always filled in (that's what I do). |
@msegers: I see, styling it though seems like an ugly hack to be honest... I added a pull that fixes the bug described above. |
…eld. (#846) * fix(MdInput): Input should not be treated as empty if it is a date field. Fixes #845 * fix(MdInput): Input should not be treated as empty if it is a date field. Fixes #845 * fix(md-input): Check for type in empty getter instead * test(MdInput): Add tests for empty check * test(MdInput): Update empty check tests to match the other tests new syntax * test(MdInput): Fix linting issue * fix(MdInput): Input should not be treated as empty if it is a date field. Fixes #845 * fix(md-input): Check for type in empty getter instead * fix(md-input): Check for type in empty getter instead * test(MdInput): Add tests for empty check * test(MdInput): Update empty check tests to match the other tests new syntax * test(MdInput): Skip empty check tests for IE11 * test(MdInput): Resolve linting issues * test(MdInput): Update empty() check tests to match new test syntax
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:
Bug
What is the expected behavior?
Placeholders should always be small for date inputs since it defaults to:
mm/dd/yyyy
.What is the current behavior?
Placeholder is large for the default value for date inputs.
What are the steps to reproduce?
http://plnkr.co/edit/MURSTrNjlRj3AGzZkEfO?p=preview
Which versions of Angular, Material, OS, browsers are affected?
The text was updated successfully, but these errors were encountered: