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

fix(input): fix invalid animation for input. #6880

Closed

Conversation

devversion
Copy link
Member

At the moment the animation will stuck if there is no md-auto-hide class or the class isn't md-input-invalid.
That's because we not trigger the callback if we finish the animation.

Fixes #6822

At the moment the animation will stuck if there is no `md-auto-hide` class or the class isn't `md-input-invalid`.
That's because we not trigger the callback if we finish the animation.

Fixes angular#6822
@ThomasBurleson
Copy link
Contributor

great catch.

@ThomasBurleson ThomasBurleson added the pr: merge ready This PR is ready for a caretaker to review label Jan 27, 2016
@ThomasBurleson ThomasBurleson added this to the 1.0.4 milestone Jan 27, 2016
@ThomasBurleson ThomasBurleson self-assigned this Jan 27, 2016
ErinCoughlan pushed a commit to ErinCoughlan/material that referenced this pull request Feb 9, 2016
At the moment the animation will stuck if there is no `md-auto-hide` class or the class isn't `md-input-invalid`.
That's because we not trigger the callback if we finish the animation.

Fixes angular#6822. Closes angular#6880
@rodwin
Copy link

rodwin commented Feb 26, 2016

Hi
Prior to this update, when going to https://material.angularjs.org/1.0.3/demo/input and do some validation, the forms below the input being validated does not slides down when the error message pops up. On 1.0.4 onward, all elements will slide down when the error message appears.

I think it is cleaner to retain the 1.0.3 version with regards to this issue.

@devversion devversion deleted the fix/input-invalid-animation branch April 19, 2016 19:53
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.

3 participants