Created an attachment (id=9913) [details]
screenshot shows huge fonts

I've seen this happen to me too. It appears to be yahoo client specific. For
example, when a yahoo contact of mine switched to IM2 I started to experience
the above scenerio. But when they switched back to trillian, the conversations
were normal again.

This used to happen before 0.10 when talking with someone using a weired
Windows client that didn't even support unicode, now it got worse, it happened
when talking with contacts using GAIM and Yahoo Messenger 6.

I get this behaviour when somebody at the other end is using a beta version of
the yahoo messenger client. I can get the exact version of the client if
required

I had the problem when the contact changed the font from Arial 10 black to
Verdana 8 red. The contact uses the latest (Whatever version that might be) of
the official Yahoo client on Windows.

Mrugesh Karnik wrote:
> I had the problem when the contact changed the font from Arial 10
> black to Verdana 8 red. 


In fact, anything on the other side not size 10 will produce huge fonts. 

Regards,
Norberto

Created an attachment (id=10983) [details]
This patch should solve this problem. It at least does against the current gaim
(1.2.1)

Does the patch breaking any of the other font handling? Please test with the 
official yahoo linux client if you can to test.  Thanks

Ok, i´ll try asap

tested with the official yahoo linux client and everything's fine.

SVN commit 413011 by mattr:

Fix the big fonts people are seeing in bug 100617.
Patch by Andre Duffeck <andre at duffeck.de>. Thanks for the patch! :)

CCBUG: 100617



 M  +2 -8      trunk/KDE/kdenetwork/kopete/protocols/yahoo/yahooaccount.cpp  


--- trunk/KDE/kdenetwork/kopete/protocols/yahoo/yahooaccount.cpp #413010:413011
@@ -639,14 +639,8 @@
 	
 	kdDebug(14180) << "Message after stripping color codes '" << newMsgText << "'" << endl;
 	
-	newMsgText.replace("<font","</font><font");
-	// if message contained <font>, remove first </font> and add </font> to end
-	int index = newMsgText.find("</font>");
-	if ( index != -1 )
-	{
-		newMsgText.remove(index, 7);
-		newMsgText.append("</font>");
-	}
+	newMsgText.replace( QRegExp("<font([^>]*)size=\"([^>]*)\"([^>]*)>"), 
+	                    QString::fromLatin1("<font\\1style=\"font-size:\\2pt\">" ) );
 	
 	kdDebug(14180) << "Message after fixing font tags '" << newMsgText << "'" << endl;
 	

SVN commit 413014 by mattr:

Fix the big fonts people are seeing in bug 100617.
Patch by Andre Duffeck <andre at duffeck dot de>. Thanks for the patch!
Backported from HEAD, should be in KDE 3.4.1

BUG: 100617




 M  +2 -8      branches/KDE/3.4/kdenetwork/kopete/protocols/yahoo/yahooaccount.cpp  


--- branches/KDE/3.4/kdenetwork/kopete/protocols/yahoo/yahooaccount.cpp #413013:413014
@@ -628,14 +628,8 @@
 	
 	kdDebug(14180) << "Message after stripping color codes '" << newMsgText << "'" << endl;
 	
-	newMsgText.replace("<font","</font><font");
-	// if message contained <font>, remove first </font> and add </font> to end
-	int index = newMsgText.find("</font>");
-	if ( index != -1 )
-	{
-		newMsgText.remove(index, 7);
-		newMsgText.append("</font>");
-	}
+	newMsgText.replace( QRegExp("<font([^>]*)size=\"([^>]*)\"([^>]*)>"), 
+	                    QString::fromLatin1("<font\\1style=\"font-size:\\2pt\">" ) );
 	
 	kdDebug(14180) << "Message after fixing font tags '" << newMsgText << "'" << endl;