Showing/hiding an emoticon causes chat text not to rearange in chat window

[xja]

After hiding a large emoticon the chat frame (in smooth operator message style) do resize but a blank space is added at the bottom of the window, so the message previuos to that with the emoticon stays stuck with its position and the following "floats" over the bottom. Here's the aspect of the "floating" message. Bye

[xja]

There's also to say that this behaviour doesn't happen when the scrollbar is present. Bye

[mathuaerknedam]

So this happens with the builtin Smooth Operator message style? What emoticon set? A particular emoticon or any of them? I need to know exactly how to replicate your problem.

[xja]

This happened with Monkey Icons Refined, the emoticon was triggered by a "mmm" within the text.
Bye

[mathuaerknedam]

I started a new chat with Smooth Operator and Monkey Icons Refined. I typed "mm" pressed enter and saw the monkey icon. I clicked on the icon and it changed to "mm". I clicked on it again and it changed back into the monkey icon. Never was there a gap. I cannot reproduce the problem, is it consistent for you?

[xja]

It's still there in b3, it happens also with smaller emoticons.
I have a MacBook (1,1) with Mac OSX 10.5.6.
bye

[mathuaerknedam]

It happens every time?

[xja]

It seems so...

[mathuaerknedam]

Aha! I can now reproduce this, but only when there is no scrollbar. Can you confirm that the problem goes away when there is a vertical scrollbar?

[mathuaerknedam]

I found the problem and I'm in the process of making a patch.

[mathuaerknedam]

patch for 1.3 and 1.4

[mathuaerknedam]

The patch does three things, though only one of them is necessary to fix this bug.

• I replaced the bits of indentation by spaces with indentation by tabs.
• I removed all instances of <tab><newline> with <newline> (in accordance with CodingStyle).
• I added to the function imageCheck:

var shouldScroll = nearBottom(); 

and

alignChat(shouldScroll); 

[catfish_man]

(In [25829]) Patch from mathuaerknedam that fixes #11602

[Robby]

Merged to branch in [25830].