412285 - Bold/Italic Arabic text renders as garbage

Status: RESOLVED FIXED

(Core :: Graphics, defect, P2)

Description

[Behnam Esfahbod [:zwnj]]

Attachment: Screenshot-IRNIC - Minefield.png

As you can see, all Bold+Italic Persian (Arabic) parts look like some random glyphs.  I'm gonna attach the render of fx2.0, and make a test case too.

Comment 1

[Behnam Esfahbod [:zwnj]]

Attachment: Screenshot-IRNIC – Mozilla Firefox.png

Correct render in Fx2.0

Comment 2

[Behnam Esfahbod [:zwnj]]

Attachment: testcase.html

Basic test case, with *very interesting* result!

Comment 3

[Behnam Esfahbod [:zwnj]]

Attachment: testcase - Minefield.png

And how the testcase renders on Jan 12th nightly build on my ubuntu 7.04 box.

Comment 4

[Behnam Esfahbod [:zwnj]]

Attachment: Screenshot-Parseek News - Minefield.jpg

Here are another screenshot (of parseek.com) which arabic text doesn't have any joining applied.  I think these two bugs have the same source.  If not, tell me to open another bug for this.

Again, nightly build 2008011204.

Comment 6

[Behnam Esfahbod [:zwnj]]

Although attached testcase works fine after removing ttf-freefont package, I still get wrong render for Bold+Italic text in http://www.nic.ir/ .  I'll try to create a new testcase.

Simon: Could it be because of another font?

Comment 7

[Behnam Esfahbod [:zwnj]]

Attachment: Screenshot-IRNIC - Minefield.png

Plain and Bold texts are OK, but Bold+Italic is rendered badly.

Also, I'm gonna report the joining problem of parseek.com separately.

Comment 8

[Behnam Esfahbod [:zwnj]]

Attachment: Screenshot-IRNIC - Persian - Minefield.png

And here are how Italic arabic text is rendered as Plain.

Comment 9

[Behnam Esfahbod [:zwnj]]

Attachment: Screenshot-IRNIC – Persian - Mozilla Firefox.png

And here is the correct render of the that page.

Comment 10

[Behnam Esfahbod [:zwnj]]

Seems the news.parseek.com problem has been solved in the upgrade, 2008011304.

Comment 11

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Font selection problem, probably.

Comment 12

[Mike Schroepfer]

Roc blocking?

Comment 13

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Yes.

Comment 14

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Might be fixed by bug 401988.

Comment 15

[Bill Gianopoulos [:WG9s]]

(In reply to comment #10)
> Seems the news.parseek.com problem has been solved in the upgrade, 2008011304.
> 
If possible, could you please try this build?

http://www.wg9s.com/mozilla/firefox/firefox-3.0b3pre.en-US.linux-i686.tar.bz2

This build includes the fix for bug 401988 which might resolve the remaining issues here.  If you could test it and report back here the results, that would be helpful.

Comment 16

[Behnam Esfahbod [:zwnj]]

(In reply to comment #15)
> If possible, could you please try this build?

I just run it, and Italic doesn't work on Arabic text at all and is ignored, so all I see is Plain and Bold text, which all are rendered correctly with DejaVu Sans.

Thanks Bill.

Comment 17

[Robert O'Callahan (:roc) (email my personal email if necessary)]

This bug is fixed now, right?

Comment 18

[Behnam Esfahbod [:zwnj]]

No.  Current trunk updates behave same as Bill's build.  Italic property is completely ignored, but Bold is fine.

Comment 19

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I think that's bug 372404. This bug was about Arabic text rendering as garbage. That doesn't happen anymore?

Comment 20

[Behnam Esfahbod [:zwnj]]

(In reply to comment #19)
> I think that's bug 372404.

Yes, seems so.

> This bug was about Arabic text rendering as garbage.
> That doesn't happen anymore?

No, it doesn't.

Thanks roc. Thanks again Bill.