Bug 3232 – CSS2: Web Kit does not support all list style types

• Component: CSS
• Status: RESOLVED FIXED

[Dave Hyatt]

Safari fails several cases of the W3C list-style-type test.

* STEPS TO REPRODUCE
1. 10.3.3, Safari 1.2.1
2. Go to http://www.w3.org/International/tests/sec-list-style-type-1.html

* RESULTS
Safari fails the following tests:
hiragana
hiragana-iroha
katakana
katakana-iroha
cjk-ideographic
georgian
armenian

* NOTES
We'll need to do some research on the last three.

[Dave Hyatt]

Apple Bug: rdar://3611345/

[Nicholas Shanks]

I have a KDE merge in the works for this, should be ready in a day or two (depending on how other 
patches are going).

[Eric Seidel (no email)]

Nick, any news on the merge patches from kde?

[Eric Seidel (no email)]

Adding mitz and ap, as this is an internationalization issue they might have interest in.

[Alexey Proskuryakov]

Hmm, looks like even CSS3 draft still doesn't list Russian as a possible list-style-type: <http://
www.w3.org/TR/2002/WD-css3-lists-20021107/>. See also: <http://www.ethiopic.org/w3c/css/WD-
css3-lists-20020220-comments.html> (I don't know whether this public note has had any effect).

[Alexey Proskuryakov]

After r18637, the results mostly match Firefox, with the following differences:

1) cjk-ideographic results look quite broken;
2) hebrew RTL rendering is different (can't tell which is correct).

[Darin Adler]

I have a patch to get us the rest of the way.

[Darin Adler]

Created attachment 12269 [details]
patch -- should also turn the test page into one or more WebKit tests

[Maciej Stachowiak]

Comment on attachment 12269 [details]
patch -- should also turn the test page into one or more WebKit tests

r=me

Make sure to land the test case with this.

[Darin Adler]

I don't want to check this fix in until the test improvements from bug 12412, since the test case is too big for a pixel test to cover it well, and unless there are list markers in the render tree dump it won't be a helpful test.

[David Kilzer (:ddkilzer)]

(In reply to comment #10)
> I don't want to check this fix in until the test improvements from bug 12412,

That's actually Bug 12142.

[Sam Weinig]

Landed in r18671.