Bug Tracker

Opened 4 years ago

Closed 4 years ago

#10606 closed bug (wontfix)

#ID-selector inconsistent

Reported by: pcvandam@… Owned by: pcvandam@…
Priority: low Milestone: None
Component: selector Version: 1.6.4
Keywords: Cc:
Blocked by: Blocking:

Description

In the theoretical case you have multiple elements on a page with the same ID, the following selectors give different results:

$('#id') $('div#id')

The first selector only returns the first occurence of the id, while the second will return a set of all elements with that id.

They should behave the same.

Change History (5)

comment:1 Changed 4 years ago by pcvandam@…

jsFiddle-example:

http://jsfiddle.net/y59Q7/

comment:2 Changed 4 years ago by rwaldron

  • Component changed from unfiled to selector
  • Owner set to pcvandam@…
  • Priority changed from undecided to low
  • Status changed from new to pending

Why would ever have >1 element with the same ID?

comment:3 Changed 4 years ago by pcvandam@…

  • Status changed from pending to new

That's not the point.

Of course it shouldn't happen, but with dynamically generated content it CAN happen. Fact is jQuery does not handle ID-selectors consistently.

comment:4 Changed 4 years ago by rwaldron

  • Status changed from new to pending

Actually, it is: http://www.whatwg.org/specs/web-apps/current-work/multipage/elements.html#the-id-attribute

If you write unexpected code, unexpected things happen.

Of course it shouldn't happen, but with dynamically generated content it CAN happen. Fact is jQuery does not handle ID-selectors consistently.

If this is an issue, the problem is not jQuery, it is the code that is dynamically generating elements that share IDs. The obvious approach is to write tests for your code that assert no duplicate IDs are created.

Last edited 4 years ago by rwaldron (previous) (diff)

comment:5 Changed 4 years ago by rwaldron

  • Resolution set to wontfix
  • Status changed from pending to closed

Just so you get a clearer picture, the "inconsistency" isn't jQuery's fault either: http://jsfiddle.net/rwaldron/JZ8GX/

Note: See TracTickets for help on using tickets.