[Pidgin] #78: Fix Bi-Directional text issues in gtkimhtml and MSN protocol

Pidgin trac at pidgin.im
Mon Apr 9 22:58:06 EDT 2007


#78: Fix Bi-Directional text issues in gtkimhtml and MSN protocol
-----------------------------------------+----------------------------------
 Reporter:  shlomil                      |        Type:  patch                                       
   Status:  new                          |    Priority:  major                                       
Milestone:  Go public with Pidgin 2.0.0  |   Component:  pidgin (gtk)                                
  Version:  2.0                          |    Keywords:  Hebrew bidi RTL Arabic "bi-directional text"
-----------------------------------------+----------------------------------
 (Patch was ported from
 [http://sourceforge.net/tracker/index.php?func=detail&aid=1668681&group_id=235&atid=300235
 patch 1668681 ] in the old BTS)

 Fixes and improvements included in this patch:

 * Added support for <SPAN style="direction:rtl;"> and <SPAN dir="RTL">
 HTML tag for the parser of IMHtml Widget.

 * Added text direction detection to outgoing messages and send it with the
 correct SPAN tag.

 * Added support for the "RL=1" flag in Messenger protocol implementation,
 both in sending and receiving messages.

 * Fixed some GTKIMHtml display issues: By injecting RLM and RLE Unicode
 characters to the conversation display according to the expected alignment
 and direction.

 * Display timestamps and nicknames in the expected order in RTL chats.

 * Fixed a problem in gtk_imhtml_get_css_opt() - Function was able to
 recognized only the
 first attribute. Now it can read all CSS attributes.

 * Fixed a problem with ICQ plug-in: The function purple_unescape_html()
 was used (oscar.c:4296) to strip HTML tags before sending ch2 messages.
 Doing that did not work so I changed it to use purple_markup_strip_html()
 .

 Discussion and screenshots are available
 [http://shlomil.blogspot.com/2007/02/patch-to-fix-some-bidi-text-issues-
 in.html here].

-- 
Ticket URL: <http://developer.pidgin.im/ticket/78>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list