Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#13084 closed defect (fixed)

Query attribute selector with square bracket fails in IE7

Reported by: Ed Siok Owned by: bill
Priority: low Milestone: 1.7.5
Component: Query Version: 1.6.1
Keywords: IE7 attribute selector query square bracket Cc:
Blocked By: Blocking:

Description (last modified by bill)

This selector always fails in IE7:

dojo.query('[name="test[0]"]')

This works in IE8 and above, along with Firefox, Chrome, and Safari

This can be seen here (if you run in IE7): http://jsfiddle.net/sioked/ma2NM/

I double checked that the jquery library works correctly in this scenario http://jsfiddle.net/sioked/GtRK2/1/

It appears as though the brackets in the selector value are causing this to fail. The brackets are common in names of input nodes/ select nodes when those nodes are used to create a unique structure on the server.

Change History (9)

comment:1 Changed 8 years ago by Ed Siok

Also tried using the dojo._base.query-sizzle via xd version of dojo. This did not work either:

http://jsfiddle.net/ma2NM/3/

comment:2 Changed 8 years ago by Adam Peller

Owner: changed from alex to Kris Zyp

comment:3 Changed 8 years ago by ben hockey

Keywords: needsreview added
Priority: highlow

comment:4 Changed 8 years ago by bill

Description: modified (diff)
Keywords: needsreview removed

Still fails in trunk (and 1.7.2) on IE6, along with 7479.

comment:5 Changed 8 years ago by bill

In [27964]:

explicit test case for #13084, refs #13084

comment:6 Changed 8 years ago by bill

Milestone: tbd1.8
Owner: changed from Kris Zyp to bill
Status: newassigned

I will check in the patch to #7479 which fixes this too.

comment:7 Changed 8 years ago by bill

Resolution: fixed
Status: assignedclosed

In [27969]:

Fix for handled quoted values in IE branch of code, fixes #7479, #13084 on IE !strict. Thanks to ptwobrussell for basis of patch.

comment:14 Changed 7 years ago by Colin Snover

In [30042]:

Fix for handled quoted values in IE branch of code, fixes #7479, #13084 on IE !strict. Thanks to ptwobrussell for basis of patch. Backport to 1.7

comment:15 Changed 7 years ago by Colin Snover

Milestone: 1.81.7.5
Note: See TracTickets for help on using tickets.