Bug Tracker

Opened 12 years ago

Closed 12 years ago

Last modified 10 years ago

#10298 closed bug (invalid)

appendChild not working in IE

Reported by: ej Owned by: ej
Priority: low Milestone: None
Component: misc Version: 1.6.4rc1
Keywords: Cc:
Blocked by: Blocking:

Description

In a in-app browser IE based the following is failing line 5644:

this.appendChild( elem );

I found #4378, and applying the same fix works:

this.parentNode.appendChild( elem );

Change History (7)

comment:1 Changed 12 years ago by addyosmani

Component: unfiledmisc
Owner: set to ej
Priority: undecidedlow
Status: newpending

Please provide us with some more information regarding the issue experienced. When you say an in-app browser IE, we'll probably assume you mean an MSIE ActiveX control unless otherwise stated. If so, what application are you referring to specifically? - there may be known issues which we can then cross-reference with other tickets. We'll need a little more information and ideally also a test case on jsFiddle that fully reproduces the behaviour you're seeing so we can evaluate it more accurately.

Last edited 12 years ago by addyosmani (previous) (diff)

comment:2 Changed 12 years ago by Andrew

This is probably not the best example, but it shows the error reported,

Run in IE7 or IE8

http://jsfiddle.net/DntAB/2/

I don't know if this is even valid, but you can see in FF/Chrome/IE9 the input will end up with "Echo!" being set as its innerHTML whereas in IE7/IE8 an error gets thrown, as it tries to append it to the input when the innerHTML call fails in the html() function.

comment:3 Changed 12 years ago by trac-o-bot

Resolution: invalid
Status: pendingclosed

Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!

comment:4 Changed 12 years ago by dmethvin

Resolution: invalid
Status: closedreopened

comment:5 Changed 12 years ago by dmethvin

Resolution: invalid
Status: reopenedclosed

The test case is attempting to set content on an input element that cannot have children. IE throws an error for this case to notify you, but other browsers ignore the invalid action. In any case it's not something that anyone should be doing.

comment:6 Changed 11 years ago by sacpundir

replace

this.appendChild( elem );

with

try
{
this.appendChild( elem );
}
catch(err)
{
}

The test case is attempting to set content on an input element that cannot have children. IE throws an error for this case to notify you, but other browsers ignore the invalid action. In any case it's not something that anyone should be doing.

Last edited 11 years ago by sacpundir (previous) (diff)

comment:7 Changed 10 years ago by cfischer83@…

IE8 does fails when trying to append to html5 elements, such as aside, section, article etc.

Note: See TracTickets for help on using tickets.