Archived Comments (locked)
|
|
The following is an archive of comments made before threaded discussions was implemented (November 16th, 2008) |
|
|
great |
|
|
actually, document.embeds exists, so you should use it instead: for(i=0;i<document.embeds.length;i++) document.embeds[i].parentNode.removeChild(document.embeds[i]) |
|
|
yes i am using xp now but when i made this script i had no idea what that meant |
|
|
ok this is one of my very old scripts
|
|
|
Ew.. You call 3 commands every time you loop. Extremely slow and inefficient. Do this...
e = document.evaluate("//embed", document, null, 6, null);
for(i=0; i<e.snapshotLength; i++){
e.snapshotItem(i).src="";
e.snapshotItem(i).width="0";
e.snapshotItem(i).height="0";
}
You need to stop using document.getElementsByTagName and use document.evaluate, and use variables correctly. Try to call as few commands as possible.
|
|
|
thanks
|
|
|
this could help in removing flash ads on any site |