new operator optional #14

Closed
basecode opened this Issue Aug 19, 2011 · 2 comments

Projects

None yet

3 participants

@basecode

What about making the new operator optional? On this point I go along with http://dmitry.baranovskiy.com/post/something-new
Anybody interested?

var opts = {
lines: 12, // The number of lines to draw
length: 7, // The length of each line

};

// Box 1
Spinner(opts).spin($(".box1"));

// Box 2
Spinner(opts).spin($(".box2"));

@LinusU
LinusU commented Aug 19, 2011

I think that this would be really good and easy to implement, +1

@fgnass
Owner
fgnass commented Aug 22, 2011

I agree with Dimitry that the box2d API sucks, but I'd really like to keep the file-size below 3000 bytes. Implementing this feature would add another 28 bytes (minified) which I'd prefer to save for something more useful.

@fgnass fgnass closed this in bc0f8b1 Sep 16, 2011
@fgnass fgnass added a commit that closed this issue Sep 16, 2011
@fgnass Merge branch 'master' into gh-pages
* master:
  Workaround for negative margin bug in IE. Fixes #27
  Changed default fps to 20
  Calling spin() now auto-stops the previous copy. Closes #28
  The new operator is now optional. Closes #14
  Fixed a typo
  Added an example page
  The frame-rate for the javascript animation fallback is now configurable.
  Added aria-role="progressbar"
  Updated the README
65f9eb8
@fgnass fgnass closed this in 65f9eb8 Sep 16, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment