Summary: | Text alignment is wrong in http://www.juniper.net/customers/support/ | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | Hasso Tepper <hasso> |
Component: | khtml | Assignee: | Konqueror Developers <konq-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | unspecified | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Attachments: | Minimal testcase |
Description
Hasso Tepper
2005-09-05 13:19:59 UTC
Created attachment 13812 [details]
Minimal testcase
Note, that problem appears with label only, if label tags are removed, it's
rendered correctly.
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: Any chance to get it backported into branch as well? |