Side navigation
#14927 closed bug (invalid)
Opened March 27, 2014 04:24PM UTC
Closed April 12, 2014 08:52AM UTC
missing argument declaration
Reported by: | Fredrik Blomqvist | Owned by: | Fredrik Blomqvist |
---|---|---|---|
Priority: | undecided | Milestone: | None |
Component: | unfiled | Version: | 1.11.0 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
the "completed" function (called from jQuery.ready.promise) seems to miss declaring argument "event" (1.10.2 has it).
function completed(/* event? */) { // readyState === "complete" is good enough for us to call the dom ready in oldIE if ( document.addEventListener || event.type === "load" || document.readyState === "complete" ) { detach(); jQuery.ready(); } }
I.e it will currently throw an error in a browser that doesn't have an addEventListener.
Attachments (0)
Change History (2)
Changed March 28, 2014 04:48PM UTC by comment:1
owner: | → Fredrik Blomqvist |
---|---|
status: | new → pending |
Changed April 12, 2014 08:52AM UTC by comment:2
resolution: | → invalid |
---|---|
status: | pending → closed |
Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!
Thanks for taking the time to contribute to the jQuery project! Please provide a complete reduced test case on jsFiddle to help us assess your ticket.
Additionally, be sure to test against the "jQuery(edge)" version to ensure the issue still exists—you may need to change this to a specific version depending on the test case.
To get you started, use the appropriate boilerplate:
Open the link and click to "Fork" (in the top menu) to begin.
Also, please read:
Additional resources: