76238 - Suggested Baltic fonts are only iso-8859-4

Status: CLOSED FIXED

(Core :: Internationalization, defect)

Description

[Gediminas Paulauskas]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux 2.4.1 i686; en-US; 0.8) Gecko/20010217
BuildID:    2001041512

In Fonts category, if I choose Baltic language encoding, all fonts are only
iso-8859-4. But everyone in Baltic countries use iso-8859-13 fonts. If Microsoft
TrueType fonts are used -- no problem, they contain all characters and can be
converted from iso-8859-4. But there are only a few normal X fonts in
iso-8859-4, while there are all main fonts in iso-8859-13.

In general, the main charset in Baltic languages is iso-8859-13, and -4 is
deprecated for a long time already.

Reproducible: Always
Steps to Reproduce:
select "Baltic" in language encoding and look into available fonts

iso-8859-13 fonts can be found at 
ftp://diedas.soften.ktu.lt/pub/linux-lt/XSEKAfonts-iso8859-13.tar.bz2

Comment 1

[Gediminas Paulauskas]

*** Bug 76235 has been marked as a duplicate of this bug. ***

Comment 2

[sairuh (rarely reading bugmail)]

->i18n?

Comment 3

[Frank Tang]

[e-mail]-
Could you try the following:
find charsetData.properties file
add one line
iso-8859-13.LangGroup               = x-baltic
after 
iso-8859-4.LangGroup               = x-baltic
and see wheater the problem go away ?

Comment 4

[Frank Tang]

mark it as moz0.9.1

Comment 5

[Gediminas Paulauskas]

strange, but Frank's suggestion does absolutely nothing. iso-8859-13 fonts
appear in User-Defined list, but not in Baltic.

Well.... at least in Mozilla 0.8. Cyrillic works fine with several charsets.

Comment 6

[marina]

re-assigning QA contact

Comment 7

[Frank Tang]

I think we also need to change nsFontMetricsGFX.cpp and add ISO-8859-13 there. 
reassign to bstell.
bstell- we talk about this this morning. 

Comment 8

[kill this account]

Attachment: patch to enable iso8859-13 fonts

Comment 9

[kill this account]

frank, would you review the patch?

Comment 10

[kill this account]

Frank,

Do you also want me to check in this change as well?

Index: intl/uconv/src/charsetData.properties
===================================================================
RCS file: /cvsroot/mozilla/intl/uconv/src/charsetData.properties,v
retrieving revision 1.20
diff -u -r1.20 charsetData.properties
--- charsetData.properties  2001/05/08 00:41:07 1.20
+++ charsetData.properties  2001/05/09 01:45:02
@@ -78,6 +78,7 @@
 iso-8859-15.LangGroup              = x-western
 iso-8859-2.LangGroup               = x-central-euro
 iso-8859-4.LangGroup               = x-baltic
+iso-8859-13.LangGroup              = x-baltic
 iso-8859-5.LangGroup               = x-cyrillic
 iso-8859-6.LangGroup               = ar
 iso-8859-6-e.LangGroup             = ar

Comment 11

[Frank Tang]

r=ftang for patch and properties file change

Comment 12

[Christopher Blizzard (:blizzard)]

I'm pretty sure that __LINE__ isn't portable.  You should use PR_LOG() or not
make the change.  Fix that and you can have an sr=blizzard

Comment 13

[kill this account]

Thanks for the sr.

It is certainly not a critical part so I'll drop it.

However, I'm curious about the portability since I see __LINE__ in the
code in 177 places:

 http://lxr.mozilla.org/seamonkey/search?string=__line__

Comment 14

[kill this account]

patch checked in

Comment 15

[Yuying Long]

Is there anyone who has Baltic fonts can verify this?
If not, Brain, could you please send the fonts to me?

Comment 16

[kill this account]

iso-8859-13 fonts can be found at 
ftp://diedas.soften.ktu.lt/pub/linux-lt/XSEKAfonts-iso8859-13.tar.bz2

Comment 17

[kill this account]

> Does anybody know  a baltic newsgroups i may subscribe to to see those 
> fonts working in mail?

Comment 18

[Gediminas Paulauskas]

server: news.omnitel.net
newsgroup: omnitel.lietuvinimas

server news.konferencijos.lt
newsgroups: lt.konferencijos.komp.lt, lt.konferencijos.komp.os.linux

I get segfault when trying to run mozilla compiled from cvs, so have not tested
this yet.

Comment 19

[marina]

changing QA contact

Comment 20

[marina]

veirification is blocked by bug # 81786

Comment 21

[marina]

Gediminas, is this Linux only bug? And what is the locale on Lituanian Linux? I 
noticed that most of those newsgoups on the omnitel server use Win-1257

Comment 22

[Gediminas Paulauskas]

Tested CVS version, the problem is fixed, pages are displayed nice with all fonts.

Marina: It should have been UNIX-like systems bug only, because in Windows fonts
are named differently, and things were displayed nice.

windows-1257 are used by Windows people and others who communicate with them,
because Microsoft products do not know anything about iso-8859-13, which is
official standard.

Comment 23

[marina]

Gediminas, thanks for the explanation