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

an element within md-select is constantly running animation #8379

Closed
demsey2 opened this issue May 8, 2016 · 3 comments
Closed

an element within md-select is constantly running animation #8379

demsey2 opened this issue May 8, 2016 · 3 comments
Assignees
Labels
has: Pull Request A PR has been created to address this issue type: bug
Milestone

Comments

@demsey2
Copy link

demsey2 commented May 8, 2016

an animation for svg element in md-select is constantly running, I am not sure if this is the best way to do it, it slows my app and many e2e tests are failing

http://codepen.io/anon/pen/bpzaYQ?editors=1010

so if locate "md-progress-circular > svh > path" element you can see that path description is constantly changing,

it only happens when md-on-open is used

here is the patch I applied to my app, it hides the issue by removing that element

https://gist.github.com/demsey2/c5e13ec09aabf8ede39c52b8987d71e6

Angular Version: 1.5.5
Angular Material Version: 11.0.-rc4

Browser Type: Chrome
Browser Version: 50
OS: Win

@stunaz
Copy link

stunaz commented May 9, 2016

patch link leads to a 404

@demsey2
Copy link
Author

demsey2 commented May 9, 2016

url fixed, btw this is not a fix

@crisbeto
Copy link
Member

crisbeto commented May 9, 2016

It's because the $$loadingAsyncDone is undefined by default, which causes the ng-if to evaluate to true.

@crisbeto crisbeto self-assigned this May 9, 2016
crisbeto added a commit to crisbeto/material that referenced this issue May 9, 2016
The loader in `md-select` is being shown/hidden via the `$$loadingAsyncDone` property. Initially it is `undefined`, which causes the loader to continue rendering in the background, until the user opens and closes the select.

Fixes angular#8379.
@crisbeto crisbeto added the has: Pull Request A PR has been created to address this issue label May 9, 2016
@ThomasBurleson ThomasBurleson modified the milestone: 1.1.1 May 18, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
has: Pull Request A PR has been created to address this issue type: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants