You should provide a selector to the on function: In that case, it will work as you expected. Also, it is better to specify some element instead of document. Read this article for better understanding: http://elijahmanor.com/differences-between-jquery-bind-vs-live-vs-delegate-vs-on/
Unless your jQuery is at the end of the document, remember to wrap it in jQuery(function($) {…}). Attach all event handlers in javaScript, not HTML. This is not essential, but allows you to see all your interactive code in a single glance. Stay aware that .play() and .pause() are <video> element methods, not jQuery methods. … Read more
Use .prop instead (and clean up your selector string): generated HTML: But the “best practices” approach is to use JavaScript event binding and this instead:
Add a second });. When properly indented, your code reads You never closed the outer $(function() {.
You get the error because order[1] is undefined. That error message means that somewhere in your code, an attempt is being made to access a property with some name (here it’s “push”), but instead of an object, the base for the reference is actually undefined. Thus, to find the problem, you’d look for code that … Read more
When you perform an DOM query through jQuery like $(‘class-name’) it actively searched the DOM for that element and returns that element with all the jQuery prototype methods attached. When you’re within the jQuery chain or event you don’t have to rerun the DOM query you can use the context $(this). Like so: $(this) will … Read more
That error can only be caused by one of three things: Your JavaScript file is not being properly loaded into your page You have a botched version of jQuery. This could happen because someone edited the core file, or a plugin may have overwritten the $ variable. You have JavaScript running before the page is … Read more
The simplest thing to do in the absence of a framework that does all the cross-browser compatibility for you is to just put a call to your code at the end of the body. This is faster to execute than an onload handler because this waits only for the DOM to be ready, not for all images … Read more
Use .prop instead (and clean up your selector string): generated HTML: But the “best practices” approach is to use JavaScript event binding and this instead:
Use setTimeout():