The default bug view has changed. See this FAQ.

Georgian list numbering is wrong

RESOLVED FIXED in mozilla1.8beta2

Status

()

Core
CSS Parsing and Computation
P1
normal
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: dbaron, Assigned: dbaron)

Tracking

({css2})

Trunk
mozilla1.8beta2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [patch], URL)

Attachments

(3 attachments)

(Assignee)

Description

12 years ago
Our Georgian list numbering is wrong.  See:
http://lists.w3.org/Archives/Public/www-international/2005JanMar/0157
http://lists.w3.org/Archives/Public/www-international/2005JanMar/0158

Steps to reproduce:
 1. load http://dbaron.org/css/test/2005/georgian-lists

Actual results: first list matches, second one doesn't
Expected results: second list matches, first one doesn't

I'm cc:ing the two people who have volunteered for Georgian localization.  I'd
appreciate if they could confirm that our current behavior is incorrect and that
we should make this change.
(Assignee)

Updated

12 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.8beta2
(Assignee)

Comment 1

12 years ago
Created attachment 178211 [details] [diff] [review]
patch
Attachment #178211 - Flags: superreview?(bzbarsky)
Attachment #178211 - Flags: review?(jshin1987)
(Assignee)

Updated

12 years ago
Keywords: css2
Priority: -- → P1
Whiteboard: [patch]

Comment 2

12 years ago
Comment on attachment 178211 [details] [diff] [review]
patch

r=jshin
Attachment #178211 - Flags: review?(jshin1987) → review+

Comment 3

12 years ago
I confirm. The Georgian numbering should indeed be lowercase.

Georgian uppercase range in Unicode is mainly resevred for old Georgisn script, 
that was used centuries ago.
(Assignee)

Updated

12 years ago
Attachment #178211 - Flags: superreview?(bzbarsky) → superreview?(roc)
Comment on attachment 178211 [details] [diff] [review]
patch

sr=bzbarsky
Attachment #178211 - Flags: superreview?(roc) → superreview+
(Assignee)

Comment 5

12 years ago
Fix checked in to trunk, 2005-03-22 15:27 -0800.
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Comment 6

12 years ago
There is no lowercase/uppercase in Georgian at all, but different Alphabets.

Character-Number correspondence scheme for Georgian is based on old Asomtavruli.
 [Unfortunately] contemporary 10E3 is missed - see http://www.gia.ge/Alphabets.html
So, sequences from 10B3(400) in source and correspondingly in patch aren't valid.
Agree with John Cowan regarding Mkhedruli - that was usual practice for medieval
Georgian script (mainly for dates).  Patch applied.

P.S. For new localization voluteer - if there are no remarks/corrections NO
confirmations needed nor for earlier, nor for this, nor for my future patches in
any lists!

Comment 7

12 years ago
Created attachment 178300 [details] [diff] [review]
Patch2

Comment 8

12 years ago
Sorry...
(Assignee)

Comment 9

12 years ago
According to the URL in comment 6, 50 and 60 are also backwards.  Should that be
corrected as well?
(Assignee)

Comment 10

12 years ago
Actually, that URL disagrees about 50 and 60 between the characters given and
the Hex values given.
(Assignee)

Comment 11

12 years ago
So, I think the following errors exist:
 * http://www.gia.ge/Alphabets.html swaps the Hex values for 50 and 60 but gets
the characters correct (I'm guessing it's that way because of the Obsolete
notation and because it's a lot easier to check the characters if you know the
language)
 * Opera gets 400 wrong (uses U+10E3 instead of U+10F3) but otherwise agrees
with http://www.gia.ge/Alphabets.html

Could you (and others?) just confirm that those two errors are as-stated (which
is what your patch does, I think) and not the other way around?
(Assignee)

Comment 12

12 years ago
Created attachment 178306 [details]
testcase
(Assignee)

Updated

12 years ago
Attachment #178300 - Flags: superreview+
Attachment #178300 - Flags: review+
(Assignee)

Comment 13

12 years ago
I committed your patch, but I'd still appreciate answers to the questions in
comment 11 so that we can improve css3-lists.

Comment 14

12 years ago
Right, HEX values missed between 50 & 60.  i check table.
U+10E3 instead of U+10F3 is "free interpretation" by Opera, but it could accepted.
Patch must be OK (You can use U+10E3 instead of U+10F3 for full compatibility
with Opera)
As i mentioned initial character-number correspondence was wrong in source - so
Your new attachment must based on right sequence of old Georgian letters.

Comment 15

12 years ago
(In reply to comment #6)
> P.S. For new localization voluteer - if there are no remarks/corrections NO
> confirmations needed nor for earlier, nor for this, nor for my future patches
> in any lists!

The issue Reporter asked for confirmation.

(Assignee)

Comment 16

12 years ago
Do any of the others here have opinions on which character is better to use for
400?  Compatibility with Opera is probably only a minor concern if we should
worry about it at all.

Comment 17

12 years ago
(In reply to comment #15)
> nor for earlier, nor for this, nor for !!!my!!! future patches in any lists!

Comment 18

12 years ago
I think we should leave it the correct way and I will have the issue reported 
and corrected to Opera Software.
You need to log in before you can comment on or make changes to this bug.