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

Inconsistent input label animation speeds #6328

Closed
apokusin opened this issue Dec 15, 2015 · 2 comments
Closed

Inconsistent input label animation speeds #6328

apokusin opened this issue Dec 15, 2015 · 2 comments
Labels
needs: review This PR is waiting on review from the team P4: minor Minor issues. May not be fixed without community contributions. pr: merge ready This PR is ready for a caretaker to review ui: CSS

Comments

@apokusin
Copy link

An input's floating label animation is quicker when using a label tag compared to just setting the placeholder attribute.

Example
tjlav5 added a commit to tjlav5/material that referenced this issue Dec 19, 2015
@tjlav5
fix(input): match up all label transitions
fe6655e 
Change all label `transitions` to be `0.25s` long, as the default is.

Fixes angular#6328
@tjlav5 tjlav5 mentioned this issue Dec 19, 2015
Closed
@tjlav5
Copy link

tjlav5 commented Dec 19, 2015

@apokusin a quick update to the CSS resulted in this: http://codepen.io/anon/pen/RradMW

Let me know if that if that fixes it for you.

Cheers

@ThomasBurleson
Copy link
Contributor

@topherfangio - can you review this with the spec and validate?

@ThomasBurleson ThomasBurleson added the needs: review This PR is waiting on review from the team label Dec 21, 2015
@topherfangio topherfangio added this to the 1.0.2 milestone Jan 6, 2016
@topherfangio topherfangio added the pr: merge ready This PR is ready for a caretaker to review label Jan 6, 2016
@ThomasBurleson ThomasBurleson added P4: minor Minor issues. May not be fixed without community contributions. and removed priority: medium labels Jan 13, 2016
@ThomasBurleson ThomasBurleson modified the milestones: 1.0.2, Backlog Jan 15, 2016
ErinCoughlan pushed a commit to ErinCoughlan/material that referenced this issue Feb 9, 2016
@ThomasBurleson @ErinCoughlan
fix(input): match up all label transitions
baa8d28 
- Thx @tjlav5

Fixes angular#6328. Closes angular#6400.
@Splaktar Splaktar removed this from the - Backlog milestone Feb 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
needs: review This PR is waiting on review from the team P4: minor Minor issues. May not be fixed without community contributions. pr: merge ready This PR is ready for a caretaker to review ui: CSS
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@topherfangio @ThomasBurleson @apokusin @tjlav5 @Splaktar