-
Notifications
You must be signed in to change notification settings - Fork 127
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
jQuery fn #6
Comments
Also, it is a bad practice to create bunch of anonymous functions on each 'toast message'. The good way is to use javascript prototypes. It could reduce memory usage by 5-15 times. |
Thanks for improving this, I was about to open an issue for the $() too. @paulmillr your link is broken @akquinet If you want people to pass in something ie. special dom element to use for a Toast message you would use something like this:
|
Thanks, did you write it in Coffeescript for kicks n' giggles or was it useful to do so? |
As it was already rewritten by me in javascript, CoffeeScript version was made just for fun, yup. |
You are doing$.fn.toastmessage which means that you have to do $ ().toastmessage. But if you remove .fn then you can remove the (). Since you are not attaching it to anything.
I think I remember seeing a way of doing both. Will have to find that out again.
The text was updated successfully, but these errors were encountered: