Chat system bugs

[Samuraid]

I installed a temporary fix into the code so you will get a nice message if the problem surfaces again. Ralle will have to look into this and I'm sure he'll have it fixed in no time.

 

[Ralle]

Thanks for covering my back Dan!
Anyway. I have fixed it now.

 

[Hakeem]

I think I've tracked down the cause of the scrolling bug:

*drum roll*

innerHTML!

It would be good to remove all instances of "innerHTML" from your code, but for now let's get rid of the major one, this should fix it:

// In your addMessage function, replace
    html = '<div style="padding-top: 1px; padding-bottom: 2px; padding-left: 1px;">'+html+'</div>';
// With
    var div = document.createElement("DIV");
    div.innerHTML = html;
    div.style.paddingTop = "1px";
    div.style.paddingBottom = "2px";
    div.style.paddingLeft = "1px";
// And
        chatWindow.innerHTML = chatWindow.innerHTML + html;
// With
        chatWindow.appendChild(div);

 

[Kwaliti]

OK, how DID you get the chat code?

 

[Pyritie]

Right-click then select View Page Source. IE can't do that. Dunno about opera, but firefox definitely can.

 

[Kwaliti]

Right-click then select View Page Source. IE can't do that. Dunno about opera, but firefox definitely can.

Oh... how stupid of me

 

[brad.dude03]

IE can view page source Pyritie dunno what you're talking 'bout lol

 

[Pyritie]

Beh, haven't used IE in such a long time. Must be a new feature or something.

 

[Ralle]

I did what Hakeem said. Please tell me if it all works now!

 

[brad.dude03]

Beh, haven't used IE in such a long time. Must be a new feature or something.

IE6 has it too, and it's not exactly new

 

[Pyritie]

IE6 has it too, and it's not exactly new

Weird.

And ralle, no scroll bugs yet!