CODEWITHSUNDEEP
javascriptinternet-explorercross-browser
buley
buley

Reputation: 29208

What are my options to avoid this error (and document.write) in Internet Explorer?

I'm maintaining some code that has far too many document.write instances. For example, the code would document.write a div, then use the DOM to find that div, then append an <img> to it.

I've tried to refactor a relevant piece to look more like this:

var node_to_append = document.createElement("div");
//node_name is a hardcoded constant
node_to_append.id = node_name;

var i = 0;
for( i = 0; i < request_urls.length; i++) {
    var image_to_append = document.createElement( "img" );
    image_to_append.width = 1;
    image_to_append.height = 1;
    image_to_append.alt = "";
    image_to_append.src = request_urls[ i ];
    node_to_append.appendChild( image_to_append );
}
//finally, append the div to the HTML doc
document.body.appendChild( node_to_append );

This is throwing an error in IE, saying "Internet Explorer can not open the Internet site http://blah.com. Operation aborted."

I'm told this is because I needed to indeed use document.write. I'm hoping there's an alternative that would allow me to use the above approach and not force me to turn my node into a string (e.g. "") and append it via document.write. Can anyone suggest a solution to my problem?

Upvotes: 1

Views: 154

Answers (1)

Ruan Mendes
Ruan Mendes

Reputation: 92274

If this code had document.writes in it, then it was being run while the page was being loaded/parsed, wasn't it? You should wrap your code in a window.onload event handler. Even better, use jQuery's ready event, or another lib's dom ready wrapper.

Upvotes: 2

Related Questions