119983 - geocities.com - Right to left text is rendered left to right.

Status: VERIFIED FIXED

(Tech Evangelism Graveyard :: English Other, defect)

Description

[Shanjian Li]

In test case of bug 118000, some of the text are specified as
right to left in direction, but rendering did not happen in that way.

Comment 1

[Simon Montagu :smontagu]

I assume I need to apply the patch from bug 118000 to view the test case.
Meanwhile, my first thought is that this could be an instance of bug 115921, if
the fix (attachment 62162 [details] [diff] [review]) is not updated in your local tree.

Comment 2

[Simon Montagu :smontagu]

Comment #1 is mistaken: this is not related to bug 115921. I think I know what
the real problem is, but this time I won't make any more half-baked suggestions
until I am quite sure.

Comment 3

[Shanjian Li]

this bug may depend on 120114. 

Comment 4

[Simon Montagu :smontagu]

There are a few different issues here.

(1) A bug similar to 115921: we don't do any bidi processing in a page that
contains neither bidi characters nor elements with dir=rtl, so on a page like
this one the RLO (right-to-left override) character is ignored. We could fix
that by adding the character to the definition of CHAR_IS_BIDI at
http://lxr.mozilla.org/mozilla/source/intl/unicharutil/public/nsIUBidiUtils.h#299,
but I'm not sure that we really need to do that: according to
http://www.unicode.org/unicode/reports/tr20/#Bidi the use of this and related
characters in markup is "strongly discouraged". We are already more liberal than
the recommendation there to ignore these characters altogether, since we process
them when bidi is enabled anyway.

(2) If the page is altered to use <bdo dir="rtl"> instead of RLO, the characters
are reversed but the display is incorrect. I assume this is bug 120114 kicking in.

Comment 5

[Simon Montagu :smontagu]

Attachment: Extract from the testcase using <bdo> instead of LRO

Comment 6

[Katsuhiko Momoi]

Evangelism contact will be made to see if that solution is
available. It may also provide additional info on whether or
not to have a fix in Mozilla.

Comment 7

[Katsuhiko Momoi]

Sent the site owner Simon's BDO suggestion along with
explanation. The owner has promised to modify the site. 
Changing Product to Evangelism, owner to myself and
accepting the bug.

Comment 8

[Katsuhiko Momoi]

There is no easy place to categorize this report.
I will put this in Europe:West since it is about
classic/extinct language in Italy. I will take this since
I am familiar with the issues inolved.
If we want to support Unicode embedding control characters
under non-bidi context, then please file a new bug.

Comment 9

[Simon Montagu :smontagu]

This has been fixed for several months now.

Comment 10

[Brant Gurganus]

v

Comment 11

[Pascal Chevrel:pascalc]

New Component