Skip to content
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

Added a new loading spinner #2582

Closed
wants to merge 2 commits into from
Closed

Added a new loading spinner #2582

wants to merge 2 commits into from

Conversation

heff
Copy link
Member

@heff heff commented Sep 11, 2015

It looks like this: http://codepen.io/heff/pen/QjjZVW

Good?

@heff heff added this to the v5.0.0 milestone Sep 11, 2015
@gkatsev
Copy link
Member

gkatsev commented Sep 12, 2015

It looks kind of weird when both the heads reach the top.

@heff
Copy link
Member Author

heff commented Sep 12, 2015

I think I see what you mean, and I think it could be because the animation didn't cover from 0 to 100%. Does it look any better now?
http://codepen.io/heff/pen/QjjZVW

@Globulopolis
Copy link

@heff yes, much better :)

@heff
Copy link
Member Author

heff commented Sep 14, 2015

Updated so the two things don't show through each other at the top.
http://codepen.io/heff/pen/ZbQePZ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants