Closed schuetzm closed 10 years ago
Pardon the super-late follow-up - I lost track of pull requests on this project quite some time ago and only update it during the winter season. ;)
Hey, no problem! It's a pleasant surprise, anyway :-) Maybe you can have a look at my other PR too?
You're most welcome. A year late is probably unusual, but not a record on GitHub, my guess. ;)
Regarding the other PR, I didn't merge that right in because there should be at least one snowStorm instance created immediately for the default case - I forgot if you were creating one right away, and had meant to review that. I was also thinking I might be able to clean up some of the events attached on each, but it's probably safe to leave the built-in functionality as-is. It should work.
One instance is always created automatically, see the last line. It's called "snowStorm", too, the same as it was before, so existing code should just keep functioning.
This allows further customization via CSS, for example setting a different cursor or transparency.