Skip to main content

Bug Tracker

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 the
    document
    . 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 gibson042 comment:1

resolution: → worksforme
status: newclosed

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.

Changed January 14, 2013 08:28PM UTC by gibson042 comment:2

component: unfiledevent

Changed January 14, 2013 08:38PM UTC by tj.vantoll 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 gibson042 comment:4

resolution: worksforme
status: closedreopened

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 gibson042 comment:5

owner: → gibson042
status: reopenedassigned

Changed January 16, 2013 05:07AM UTC by mikesherov 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 Richard Gibson comment:7

resolution: → fixed
status: assignedclosed

Fix #13208: only check elements for delegation matches

Changeset: fc2ba2e1361126c39f955437ee025cfca3bffa65

Changed January 16, 2013 05:21AM UTC by gibson042 comment:8

milestone: None1.9.1

Changed February 12, 2014 10:21PM UTC by dcherman 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 ):

http://jsfiddle.net/nsatZ/3/

Changed February 21, 2014 02:42AM UTC by dmethvin 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 navjot.singh@allscripts.com comment:11

I am having the same problem. Can you tell the solution to this problem ?