Improve CSS3 Glyphs Support for lists

• Component: CSS
• Status: UNCONFIRMED ---

[Adam M-W]

The CSS3 Definition defines glyphs for box, check, circle, diamond, disc, hyphen and square.
These glyphs should be added to both the text security properties and CSS list values (see /WebCore/rendering/RenderText.cpp and /WebCore/rendering/RenderListMarker.cpp)

Currently, circle, disc and square are already implemented which leaves box (U+25FD), check (U+2713), diamond (U+25C6) and hyphen (U+2043) to be implemented.

Further information can be found at http://www.w3.org/TR/2002/WD-css3-lists-20021107/#glyphs

[Alexey Proskuryakov]

Updating list-style-type properties sounds fine, although I'm not sure if a spec from 2002 should be considered authoritative. Do other browsers support all those values?

As for -webkit-text-security, it doesn't seem related to me. Is there any indication anywhere that it should support that same glyphs as list-style-type?

[Adam M-W]

Some other CSS3 that's even older has already been implemented, why wait longer?
As long as if the recommendation changes (which it probably won't in regard to glyphs as there are NO comments on it so far) we update WebKit to match, i don't see a problem. WebKit already uses stuff like CSS3 Backgrounds which are WD so should be fine.

I've just done a quick check now, and it doesn't seem that other browsers support it (tested FF 3.5, IE 8) but that shouldn't stop us, as it won't ruin the default behaviour only improve support where specified.

I've recommended webkit-text-security be updated as well as the current code references the glyphs from the text-security module as they are used there as well, so doing so for these new glyphs allows for code to be re-used and similar to existing code, and also gives new webkit-specific features in terms of new symbols (can't hurt).

I thought that this should be a realitivly simple task as most code can be copied and renamed, but I couldn't find where they were set in text-security so I couldn't create a patch. (ergo the bug report / feature request).

(Also I've got no idea how to make testcases so I'd thought I'd leave it to the experts)

[Daniel Bates]

For your reference, this bug is related to bug #32138 ("Upper-greek list-style not supported"). We also landed a cleanup bug #32799 to clear the way for adding more list styles.

[Brent Fulgham]

Renaming to just cover the Glyph support.

[Radar WebKit Bug Importer]

<rdar://problem/96904988>