Side navigation
#13208 closed bug (fixed)
Opened January 14, 2013 04:39PM UTC
Closed January 16, 2013 05:16AM UTC
Last modified February 21, 2014 08:28AM UTC
JS error trying to get the id of the document in IE8
Reported by: | tj.vantoll | Owned by: | gibson042 |
---|---|---|---|
Priority: | undecided | Milestone: | 1.9.1 |
Component: | event | Version: | git |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
See http://jsfiddle.net/tj_vantoll/tg2VU/ and click on the link in IE8.
I get a JS error from this block in Sizzle:
Expr.filter["ID"] = function( id ) { var attrId = id.replace( runescape, funescape ); return function( elem ) { //The error occurs here when trying to call getAttribute //on the document itself return elem.getAttribute("id") === attrId; }; };
Couple of things:
- There needs to be an
id
listener attached to thedocument
. It doesn't need to do anything, but it must be there. - This seems specific to tables, if I switch the
<table>
with a<div>
the problem does not occur. - The issue does not occur in IE7, IE9, or the latest Chrome and Firefox. It seems specific to IE8.
I was unable to debug this any deeper on my own.
Thanks.
Attachments (0)
Change History (11)
Changed January 14, 2013 08:27PM UTC by comment:1
resolution: | → worksforme |
---|---|
status: | new → closed |
Changed January 14, 2013 08:28PM UTC by comment:2
component: | unfiled → event |
---|
Changed January 14, 2013 08:38PM UTC by comment:3
It's an IE8 only issue. I see it on http://jsfiddle.net/tj_vantoll/tg2VU/show in both a real IE8 install as well as IE8 via BrowserStack.
The link is successfully removed but I do get a JS error.
Changed January 16, 2013 04:50AM UTC by comment:4
resolution: | worksforme |
---|---|
status: | closed → reopened |
Ah, I see now. Removing the ancestor element makes its parent a DocumentFragment
for further event processing, which is invalid input for Sizzle. Sneaky.
Changed January 16, 2013 04:51AM UTC by comment:5
owner: | → gibson042 |
---|---|
status: | reopened → assigned |
Changed January 16, 2013 05:07AM UTC by comment:6
Tsk tsk. You should know better than to challenge the king of all triagers: tj.vantoll. :)
Changed January 16, 2013 05:16AM UTC by comment:7
resolution: | → fixed |
---|---|
status: | assigned → closed |
Fix #13208: only check elements for delegation matches
Changeset: fc2ba2e1361126c39f955437ee025cfca3bffa65
Changed January 16, 2013 05:21AM UTC by comment:8
milestone: | None → 1.9.1 |
---|
Changed February 12, 2014 10:21PM UTC by comment:9
This seems like it may still be an issue in the 2.X branch. Here's a repro case ( tested in Chrome Latest ):
Changed February 21, 2014 02:42AM UTC by comment:10
dcherman can you open a new ticket? Tagging onto old tickets is not good, even if it might be the same problem. Thanks!
Changed February 21, 2014 08:28AM UTC by comment:11
I am having the same problem. Can you tell the solution to this problem ?
I am unable to reproduce on jquery-1.8.3 or jquery-git with IE7 or IE8. Please let us know if you continue to observe this behavior.