Skip to main content

Bug Tracker

Side navigation

#9905 closed bug (fixed)

Opened July 25, 2011 05:04PM UTC

Closed December 28, 2012 02:24AM UTC

.removeAttr( "id" ) sometimes crashes IE 7

Reported by: scottgonzalez Owned by:
Priority: high Milestone: 1.9
Component: attributes Version: 1.6.2
Keywords: ie7 Cc:
Blocked by: Blocking:

I can't really describe this any better than the raw code from the test case does but...

If you clone an element and then call .removeAttr( "id" ) everything is fine. If you repeat the cloning and removal of the attribute, everything is still fine. However, if you first query for one of its ancestors, and then query from there for a descendant, IE 7 will crash.

I haven't attempted to reduce this any further, e.g., querying for a non-existent descendant, or querying by something other than id.

This exists in 1.6.x and master and seems to only affect IE 7 (even IE 6 is fine).

Test case:

Attachments (0)
Change History (7)

Changed July 25, 2011 05:09PM UTC by scottgonzalez comment:1

jQuery UI ticket with workaround:

Changed July 25, 2011 06:09PM UTC by timmywil comment:2

component: unfiledattributes
priority: undecidedhigh

That's strange.

Changed July 25, 2011 06:09PM UTC by timmywil comment:3

status: newopen

Changed July 26, 2011 03:02PM UTC by comment:4

Thank goodness somebody else has found this as I was thinking I was going crazy. I definitely can repeat this:

options.templates.tabs = options.tabsTemplateRoot.clone()
alert('clone complete');
alert('class removed');
alert('id removed');

On second time around Internet Explorer 7 only gets to "alert class removed" and then crashes. At least for the moment we can work around.

Changed March 03, 2012 08:56PM UTC by dmethvin comment:5

keywords: → ie7

I suspect this is related to #9646.

Changed December 04, 2012 05:54AM UTC by gibson042 comment:6

milestone: 1.next1.9

Batch update of oldIE bugs to milestone 1.9 for resolution/reevaluation/closing/etc.

Changed December 28, 2012 02:24AM UTC by dmethvin comment:7

resolution: → fixed
status: openclosed

Fixed with the commit for #9646.