Great hack...but found a problem. It will not work in IE at least with the Ultimate.cgi version of the hack.
Netscape can handle multiple BODY statements in the HTML. IE cannot, it'll toss the second one. So, we need to get that onLoad= bit into the first body statement and eliminate the second one.
This is my first time documenting this...so, I hope this helps.
So to fix this, we eliminate the old $pmpop mess by removing it from the bottom of the code. These lines go away:
$CurrentPMpop = "$cookie{pmpop}";
if ($CurrentPMpop eq "1") {
$pmpop = qq(); }
else {
$pmpop = qq(VLINK="#000080" ALINK="#000080" onLoad="poppm()>");
}
Also, at the very bottom code in one of the subroutines, there's a $pmpop on a line by itself inside a print<< routine. We kill that too. It's just after the users online stuff prints.
Now, back up in the code we find the first BODY statement inside a large "print << INTROHTML" routine.
Make it look like this...
And that is it. It will now work in both IE and Netscape because this will setup only one BODY tag based on if they have messages or not. If no messages, the standard body tag is used, if new messages, it will add in the onload call.
I thank my friend Otto for helping me with this for I am learning as I go.
------------------
David Bott
AVS Forum Admin
http://www.avsforum.com This message has been edited by David Bott on October 22, 2000 at 08:36 PM
