Bug 11469

Summary:

[S60] Squares are displayed instead of characters for Greek web pages

Product:

WebKit

Reporter:

Zhaosong Lu <Zhaosong.Lu>

Component:

WebKit Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED FIXED

Severity:

Critical

CC:

bradley.morrison, shyam.sareen

Priority:

Version:

420+

Hardware:

S60 Hardware

OS:

S60 3rd edition

URL:

http://www.google.com

Attachments:

Description	Flags
Fix of the bug	none
Patch for 3.2	yongjun.zhang: review+

Zhaosong Lu

Reported 2006-10-31 11:05:13 PST

For the Variant 7 build, if the phone language is set to Greek, squares are displayed instead of characters.

Attachments
Fix of the bug (2.34 KB, patch) 2006-10-31 14:24 PST, Zhaosong Lu	no flags	Details Formatted Diff Diff
Patch for 3.2 (2.25 KB, patch) 2007-02-08 14:23 PST, Shyam Sareen	yongjun.zhang: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Zhaosong Lu

Comment 1 2006-10-31 14:24:18 PST

Created attachment 11312 [details] Fix of the bug Browser defaults font family to "Nokia Sans S60" which is not supported under variant 7 build. The system has to default to another font family. If the phone language is English, the system defaults the font family to "Series 60 Hindi" which contains fonts for Greek. If the phone language is set to Greek, the system default the font to "Series 60 ZDigi" which does not contains fonts for Greek. That is why the same variant 7 build supports Greek under English and Automatic Language settings while it does not under Greek language setting. My fix is setting the font family to "Series 60 Hindi" if the phone language is set to Geek language. I have tested with hardware on Variant 7 build and everything works fine.

Yongjun Zhang

Comment 2 2006-11-02 07:57:54 PST

Comment on attachment 11312 [details] Fix of the bug r=me

Yongjun Zhang

Comment 3 2006-11-02 08:46:45 PST

commited: r17537

Sachin Padma

Comment 4 2006-11-02 13:56:53 PST

3.1m -> 17544

Shyam Sareen

Comment 5 2007-02-08 14:23:46 PST

Created attachment 13067 [details] Patch for 3.2 The last patch worked for 3.1m but it is not working for 3.2. Before setting the new fonts in the text renderer, check that font is available. If fonts are not available then leave the font as default font.

Bradley Morrison

Comment 6 2007-02-09 10:52:53 PST

new patch so re-opening (you can re-opend bugs now btw :)

Bradley Morrison

Comment 7 2007-02-09 11:10:58 PST

Comment on attachment 11312 [details] Fix of the bug clearing review flag (this has been landed)

Yongjun Zhang

Comment 8 2007-02-09 13:40:14 PST

Comment on attachment 13067 [details] Patch for 3.2 r=me

Bradley Morrison

Comment 9 2007-02-12 07:07:37 PST

(In reply to comment #8) > (From update of attachment 13067 [details] [edit]) > r=me > Landed - 19574, & resolved.

Krishna

Comment 10 2007-02-19 17:11:57 PST

Verified and Closed.

Note You need to log in before you can comment on or make changes to this bug.