75009 - Bidi: Unicode Arabic text in table not connected

Status: RESOLVED INVALID

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

Description

[Maha Abou El-Rous]

From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 5.01; Windows 98)
BuildID:    00000000 - Bidi build dated 20010404

Arabic Unicode text is displayed disconnected (not properly shaped) in file 
Ken.htm

Reproducible: Always
Steps to Reproduce:
1.Open file Ken.htm


Actual Results:  Arabic text is displayed disconnected all in Middle character 
shapes

Expected Results:  View same file on IE5 for correct character shapes

Comment 1

[Maha Abou El-Rous]

Attachment: Unicode text in table

Comment 2

[Frank Tang]

focus on landing IBMBIDI without breaking Latin and CJK on moz0.9. move bidi 
functional bug to moz0.9.1

Comment 3

[Frank Tang]

erik resign. reassign all his bug to ftang for now.

Comment 4

[Frank Tang]

mark it as assign

Comment 5

[Frank Tang]

decommit other bidi issue until we land IBMBIDI default on.

Comment 6

[Frank Tang]

reassign to [e-mail]

Comment 7

[Andreas Becker]

Changing QA contact to [e-mail].

Comment 8

[Jaime Rodriguez, Jr.]

Ftang - Is this a beta stopper, or will Arabic support not be complete for nsbeta1?

Comment 9

[Jaime Rodriguez, Jr.]

Marking as nsbeta1-. Arabic support will not be in until M0.9.2 window

Comment 10

[Jaime Rodriguez, Jr.]

Assigning M0.9.2 | P2. I think we need this for RTM.

Comment 11

[Maha Abou El-Rous]

Setting dependency on Shaping (82383) to have correct shaping and Bidi UI 
(85420) to set text type visual/logical for correct order.

Comment 12

[Jaime Rodriguez, Jr.]

Ftang - Does it look like Arabic will be supported in the final release?

Comment 13

[Simon Montagu :smontagu]

I'm not sure why this has a milestone but is dependent on bugs that don't.
Resetting component while I'm here.

Comment 14

[Zach Lipton [:zach]]

Mass-move all BiDi Hebrew and Arabic qa to me, [e-mail]. 
Thank you Gilad for your service to this component, and best of luck to you 
in the future.

Sholom. 

Comment 15

[Katsuhiko Momoi]

arabic contact to maha.

Comment 16

[Simon Montagu :smontagu]

Attachment: Reduced testcase in logical UTF-8

Comment 17

[Simon Montagu :smontagu]

Can someone confirm whether the testcase in attachment 48281 [details] is in fact valid?
It seems to me that the characters are wrong in the source file. For example the
word for "Arabic" in the "Native Language" column is always spelt with character
FE90; ARABIC LETTER BEH FINAL FORM, but I believe it should be FE91; ARABIC
LETTER BEH INITIAL FORM.

Apart from that and some other cases, the display seems correct to me (tested on
Bidi enabled and US Windows NT), and I think this can be marked FIXED.

Comment 18

[Maha Abou El-Rous]

True there are some characters having the wrong shapes, but this was not the problem 
reported. the output was displayed in a totally different manner than IE5 which was the 
reference in this case. IE5 handles UTF-8 as a visual charset. we need to define it as 
such.
Adjusting dependency on 81664 to have correct display on Bidi & non-Bidi systems.
Attaching a fix to define UTF-8 as visual charset.  

Comment 19

[Maha Abou El-Rous]

Attachment: patch to set UTF-8 as a visual codepage

Comment 20

[Simon Montagu :smontagu]

Maha: This bug is separate from the problem with the ordering of UTF-8, which is
bug 75004. That was closed as invalid after I commented:

>The Arabic Hebrew and Farsi text in this example are in visual order (which is
>incorrect for UTF-8) so Mozilla's behaviour here is correct.
>
>On selecting Bidi Options | Text Type | Visual, the text is displayed as >expected.
>
>The Arabic and Farsi are displayed correctly in IE5 because two errors are
>cancelling one another out: the visual order and the fact that IE5 doesn't
>classify Bidi characters in the FExx range as right-to-left. Note that the
>Hebrew text is displayed reversed in IE5 and AFAIK there is no way in the UI to
>correct this.

Notice that UTF-8 pages with Arabic in the 06xx range are displayed as logical
in both IE and Mozilla, for example http://www.unicode.org/iuc/iuc10/x-utf8.html

Comment 21

[Maha Abou El-Rous]

Simon, you are right, the example got me confused. 
As far as this bug is concerned we only need patch in bug# 81664 in order to have similar 
display on Bidi & non-Bidi systems. Please disregard the latest patch I attached.
We also need the Bidi options present so we have correct display by forcing the TextType 
visual, adding dependency on #85420

Comment 22

[Ahmad A. Abu-Taha]

Kindly be informed that Ahmad A. Abu-Taha ([e-mail]) from IBM Egypt is 
replacing Maha Abou El-Rous ([e-mail]) in monitoring and receiving 
notifications of Mozilla bugs regarding Arabic.

Comment 23

[Uri Bernstein]

Resolving as INVALID given the comments above (and the fact that we no longer support Win98, and that if there ever was a real problem reported here, it's long gone).