Bug Tracker

Modify ↓

Ticket #5172 (closed enhancement: invalid)

Opened 4 years ago

Last modified 2 years ago

Make the context parameter behave more consistently

Reported by: lrbabe Owned by:
Priority: minor Milestone: 1.4
Component: core Version: 1.3.2
Keywords: needsreview Cc:
Blocking: Blocked by:

Description

Currently the three following syntax do not result in the same context parameter:

$("div", document.getElementById("myId")); $("div", $("#myId")); $("div", "#myId");

Only the first one result in the context property to be properly set. The attached patch fix this issue. See related thread:  http://groups.google.com/group/jquery-dev/browse_thread/thread/b8c414b373163895

Attachments

patchContext.txt Download (570 bytes) - added by lrbabe 4 years ago.

Change History

Changed 4 years ago by lrbabe

comment:1 Changed 3 years ago by dmethvin

  • Keywords needsreview added; context, consitency removed
  • Status changed from new to open

comment:2 Changed 2 years ago by dmethvin

  • Status changed from open to closed
  • Resolution set to invalid

This patch has gone stale since now the user can pass in a plain object full of attributes. I'm not convinced it's necessary anyway.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View ↑

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.