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

Add spin.js in favor of a high-res spinner #180

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mafigit
Copy link

@mafigit mafigit commented Feb 27, 2014

  • spin.js provides a spinner with a better resolution than the old gif
    spinner. This especially comes in handy on retina displays

- spin.js provides a spinner with a better resolution than the old gif
  spinner. This especially comes in handy on retina displays
@mafigit
Copy link
Author

mafigit commented Feb 28, 2014

I was not sure if I should ship the 'compiled' css and js files.

@lokesh lokesh added the feature label Mar 17, 2014
@lokesh lokesh force-pushed the master branch 3 times, most recently from de83cee to c1c0f87 Compare June 16, 2015 04:14
@johansmitsnl
Copy link

Should it not be better to use an icon font with spinner functionality instead of images? #556

@lokesh lokesh added this to the 3.0 milestone Apr 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants