[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 112073] Text alignment is wrong in
From:       Germain Garand <germain () ebooksfrance ! com>
Date:       2007-01-28 20:06:45
Message-ID: 20070128200645.10289.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=112073         
germain ebooksfrance com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From germain ebooksfrance com  2007-01-28 21:06 -------
SVN commit 627993 by ggarand:

only allow spilling of line boxes if it's consistent with current 
directionality.
(cherry picked in Webcore's r7692 from David Hyatt <hyatt apple com>)

BUG: 112073



 M  +4 -1      bidi.cpp  


--- trunk/KDE/kdelibs/khtml/rendering/bidi.cpp #627992:627993
 @ -729,6 +729,8  @
     switch(style()->textAlign()) {
         case LEFT:
         case KHTML_LEFT:
+            if (style()->direction() == RTL && totWidth > availableWidth)
+                x -= (totWidth - availableWidth);
             numSpaces = 0;
             break;
         case JUSTIFY:
 @ -742,7 +744,8  @
                 break;
         case RIGHT:
         case KHTML_RIGHT:
-            x += availableWidth - totWidth;
+            if (style()->direction() == RTL || totWidth < availableWidth)
+                x += availableWidth - totWidth;
             numSpaces = 0;
             break;
         case CENTER:
[prev in list] [next in list] [prev in thread] [next in thread]