Bug Tracker

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#3666 closed enhancement (fixed)

Remove dead code from $(fn) shortcut for $(document).ready(fn)

Reported by: geary Owned by: flesler
Priority: trivial Milestone: 1.3
Component: core Version: 1.2.6
Keywords: Cc: geary
Blocked by: Blocking:

Description

The code for the $(fn) shortcut is:

return jQuery( document )[ jQuery.fn.ready ? "ready" : "load" ]( selector );

It's unclear what the intent of this code was, but the jQuery.fn.ready test has no effect. jQuery.fn.ready always exists, so the code is the equivalent of:

return jQuery( document ).ready( selector );

That also matches people's expectation that $(fn) is the exact equivalent of $(document).ready(fn).

The attached patch removes this dead code.

Attachments (1)

readyshortcut.patch (402 bytes) - added by geary 9 years ago.
Patch to remove dead code from $(fn)

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by geary

Attachment: readyshortcut.patch added

Patch to remove dead code from $(fn)

comment:1 Changed 9 years ago by flesler

Cc: geary added
Owner: set to flesler
Status: newassigned

comment:2 Changed 9 years ago by flesler

Resolution: fixed
Status: assignedclosed

Applied at [5997]. Thanks.

comment:3 Changed 9 years ago by flesler

Type: bugenhancement
Note: See TracTickets for help on using tickets.