Closed Bug 985825 Opened 10 years ago Closed 10 years ago

Unprefix -moz-ethiopic-numeric

Categories

(Core :: CSS Parsing and Computation, enhancement, P5)

enhancement

Tracking

()

RESOLVED FIXED
mozilla33

People

(Reporter: xidorn, Assigned: xidorn)

References

(Blocks 2 open bugs)

Details

(Keywords: css3, dev-doc-complete)

Attachments

(1 file, 1 obsolete file)

-moz-ethiopic-numeric could be unprefixed since this counter style has been defined in CSS Counter Styles spec, and it seems to be working correctly.
Attached patch patch (obsolete) — Splinter Review
Assignee: nobody → quanxunzhen
Status: NEW → ASSIGNED
Attachment #8439877 - Flags: review?(jfkthame)
Comment on attachment 8439877 [details] [diff] [review]
patch

Review of attachment 8439877 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, thanks! A couple of (very minor) suggestions below.

::: layout/style/nsStyleConsts.h
@@ +654,5 @@
>  #define NS_STYLE_LIST_STYLE_SIMP_CHINESE_FORMAL   16
>  #define NS_STYLE_LIST_STYLE_TRAD_CHINESE_INFORMAL 17
>  #define NS_STYLE_LIST_STYLE_TRAD_CHINESE_FORMAL   18
>  #define NS_STYLE_LIST_STYLE_MOZ_TAMIL             19
> +#define NS_STYLE_LIST_STYLE_ETHIOPIC_NUMERIC      20

Maybe swap MOZ_TAMIL and ETHIOPIC_NUMERIC around, to match the order they're used in nsCSSProps.cpp? It shouldn't really matter, just seems tidier to me.

::: layout/style/test/property_database.js
@@ +2681,5 @@
>        "-moz-oriya", "-moz-kannada", "-moz-malayalam", "-moz-bengali",
>        "-moz-tamil", "-moz-telugu", "-moz-thai", "-moz-lao",
>        "-moz-myanmar", "-moz-khmer",
>        "-moz-hangul", "-moz-hangul-consonant",
> +      "-moz-ethiopic-halehame", "ethiopic-numeric",

Looking at the values here, the convention seems to be that the standard ones come first, and then the -moz-prefixed extras; which suggests moving ethiopic-numeric up among the standard values.

Also, should we keep the prefixed value here in addition to the non-prefixed? That seems to be what's been done for -moz-japanese-*, for example. WDYT?
Attachment #8439877 - Flags: review?(jfkthame) → review+
https://tbpl.mozilla.org/?tree=Try&rev=4ba413620f67

This patch should be applied after the patch for bug 982355.
Attachment #8439877 - Attachment is obsolete: true
Attachment #8440210 - Flags: review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/c3475bd79048
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.