148866 - align=justify not working well with direction=rtl

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, defect)

Description

[Tomer Cohen :tomer]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.0rc2)
Gecko/20020510
BuildID:    20020523

Like bug 146135 ["align=justify cases rtl text to be jumbled" - under Mac], I
got something similar under win. the font is readable (The other bug make it
unreadable, as you can see in that bug attachment. This bug simply ignore the
justify property.

(I didn't published this in the same bug, because it provide other result, but
it's possible I'm wrong, and this is another variation of the same bug.)

Reproducible: Always

Tested under RC3 and 0.9.5 - The same goes there

Comment 1

[Tomer Cohen :tomer]

Attachment: Test case

This is example how to reproduce the problem.

Comment 2

[Shoshannah Forbes]

Can you attach a screen shot please? I don't have here mozilla under win2k, but 
from the test case this sounds similar to bug 131023  as well (althugh there is 
no div's in your code).

p.s.
For a few months now, planet nana is not reachable outside of Israel (I see you 
put a copy there as well)

Comment 3

[Tomer Cohen :tomer]

Attachment: Screen shot of the bug in action

Comment 4

[Tomer Cohen :tomer]

Shoshana: I've already made screenshot (but probably forgot to commit it).
Anyway, the attachment now online. It's looks like bug 131023, but my problem is
mainly with justify, not divs. You can see the problem in the attachment.

Comment 5

[Dave]

How about Bug 146135?

Comment 6

[Simon Montagu :smontagu]

This does look like either a dupe or a close relation of bug 131023 and/or bug
121633. What all these bugs have in common is the combination of margins and
right-to-left direction.

Comment 7

[Simon Montagu :smontagu]

This currently WORKSFORME in trunk (but is still broken in branch). Tomer, can
you retest with a current nightly?

Comment 8

[Simon Montagu :smontagu]

Sorry, ignore my last comment. I was testing with my local build, not a trunk build.

Comment 9

[Simon Montagu :smontagu]

This is fixed by David Baron's fix to bug 131023

Comment 10

[(no longer active)]

Mass-assigning the new rtl keyword to RTL-related (see bug 349193).