Bug Tracker

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#5652 closed bug (fixed)

focus(), blur(), and submit() no longer working in 1.4

Reported by: ixpleo Owned by:
Priority: critical Milestone: 1.4
Component: event Version: 1.4a2
Keywords: Cc:
Blocked by: Blocking:

Description

When testing 1.4a1 in a live application (dev version is running 1.3.2), focus(), blur(), and submit() no longer work.

I created a barebones, test html file to verify. There is a form with a hidden input in the body, and these were the tests:

using document.getElementById().[focus|blur|submit]() works just fine, but not $("#inputIDHere").[focus|blur|submit]()

I haven't tested other events to verify whether or not they are working - just blur(), focus(), and submit()

Attachments (2)

ticket5652.html (2.3 KB) - added by ixpleo 10 years ago.
Test File Showing focus(), blur(), and submit() bug
native.patch (541 bytes) - added by adamlogic 10 years ago.
This patch fixes the issue for me, but I don't know what side effects might be. I have not run the test suite against this patch.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 10 years ago by dmethvin

Resolution: invalid
Status: newclosed

Please reopen and attach or link to your test file. Thanks!

Changed 10 years ago by ixpleo

Attachment: ticket5652.html added

Test File Showing focus(), blur(), and submit() bug

comment:2 Changed 10 years ago by ixpleo

Resolution: invalid
Status: closedreopened

Reopened and attached test file showing jQuery bug for focus(), blur(), and submit()

Also found a blog post online using 1.4a1 that documents the same problem with blur in 1.4a1:

http://www.bennadel.com/index.cfm?dax=blog:1772.view

Here is the relevant portion of his post:

Reset and focus the first input element. NOTE: There appears to be a bug here where jQuery cannot use its own FOCUS method. I had to get the raw NODE to do that. newForm.find( ":text" )

.val( "" )

.get( 0 ).focus()

;

comment:3 Changed 10 years ago by dmethvin

From the test case, it looks like this still works for IE but not Firefox, Opera, or Safari.

Changed 10 years ago by adamlogic

Attachment: native.patch added

This patch fixes the issue for me, but I don't know what side effects might be. I have not run the test suite against this patch.

comment:4 Changed 10 years ago by adamlogic

Attached a potential fix.

comment:5 Changed 10 years ago by adamlogic

Ignore my fix. This appears to be working in 1.4a2.

comment:6 Changed 10 years ago by ixpleo

I've tested the attached test case with 1.4a2 and can confirm this bug has been fixed (as adamlogic mentioned).

comment:7 Changed 10 years ago by john

Resolution: fixed
Status: reopenedclosed
Version: 1.4a11.4a2

comment:8 Changed 10 years ago by john

Component: unfilledevent
Note: See TracTickets for help on using tickets.