Bug 11469 - [S60] Squares are displayed instead of characters for Greek web pages
Summary: [S60] Squares are displayed instead of characters for Greek web pages
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 420+
Hardware: S60 Hardware S60 3rd edition
: P2 Critical
Assignee: Nobody
URL: http://www.google.com
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-31 11:05 PST by Zhaosong Lu
Modified: 2007-02-19 17:11 PST (History)
2 users (show)

See Also:

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 an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Zhaosong Lu 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.
Comment 1 Zhaosong Lu 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.
Comment 2 Yongjun Zhang 2006-11-02 07:57:54 PST 
Comment on attachment 11312 [details]
Fix of the bug

r=me
Comment 3 Yongjun Zhang 2006-11-02 08:46:45 PST 
commited: r17537
Comment 4 Sachin Padma 2006-11-02 13:56:53 PST 
3.1m -> 17544
Comment 5 Shyam Sareen 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.
Comment 6 Bradley Morrison 2007-02-09 10:52:53 PST 
new patch so re-opening (you can re-opend bugs now btw :)
Comment 7 Bradley Morrison 2007-02-09 11:10:58 PST 
Comment on attachment 11312 [details]
Fix of the bug

clearing review flag (this has been landed)
Comment 8 Yongjun Zhang 2007-02-09 13:40:14 PST 
Comment on attachment 13067 [details]
Patch for 3.2

r=me
Comment 9 Bradley Morrison 2007-02-12 07:07:37 PST 
(In reply to comment #8)
> (From update of attachment 13067 [details] [edit])
> r=me
> 

Landed - 19574, & resolved.
Comment 10 Krishna 2007-02-19 17:11:57 PST 
Verified and Closed.