jquery-simulate

jquery-simulate copied to clipboard

Very useful project! I find that when running tests with jasmine-maven-plugin, specifically when running a headless test (which internally uses HtmlUnit for the "browser") that event.relatedTarget can be null. This...

lukepalmer

Simulating click should simulate mousedown, mouseup, click

1

kborchers

See **1. Linting** in the [JS style guide](http://contribute.jquery.org/style-guide/js/).

scottgonzalez

mikesherov

currently it's passed in type as a formal parameter: ``` dispatchEvent: function( elem, type, event ) ``` can't we just use `event.type`?

mikesherov

This code assumes keyCode can be less than zero, is that even true? ``` event.keyCode = (options.charCode > 0) ? options.charCode : options.keyCode; ``` can we just use: ``` is...

mikesherov

currently, we fallback from ``` event = document.createEvent( "KeyEvents" ); event.initKeyEvent ``` to just creating a generic event in Webkit. Webkit has an alternate implementation called `initKeyBoardEvent`, which needs to...

mikesherov

currently, only event.button is populated. However, there is now also event.buttons in some browsers.

mikesherov

currently, the clientX/Y is set to 1 for the mouseEvent if not overridden by options. This should default to a position within the target element.

mikesherov