Closed Bug 774335 Opened 8 years ago Closed 8 years ago

Implement unicode-bidi: isolate-override (replacing ability to combine isolate and bidi-override keywords)

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: smontagu, Assigned: smontagu)

References

()

Details

(Keywords: dev-doc-complete)

Attachments

(2 files)

The latest version of CSS3 Writing Modes has a new value "isolate-override" for unicode-bidi, instead of the combination "bidi-override isolate".
Attached patch PatchSplinter Review
Assignee: nobody → smontagu
Attachment #642961 - Flags: review?(dbaron)
Comment on attachment 642961 [details] [diff] [review]
Patch

>diff --git a/layout/style/nsCSSPropList.h b/layout/style/nsCSSPropList.h

> CSS_PROP_TEXTRESET(
>     unicode-bidi,
>     unicode_bidi,
>     UnicodeBidi,
>-    CSS_PROPERTY_PARSE_VALUE |
>-        CSS_PROPERTY_VALUE_PARSER_FUNCTION,
>-    "",
>-    0,
>+    CSS_PROPERTY_PARSE_VALUE,
>+    VARIANT_HK,
>     kUnicodeBidiKTable,
>     CSS_PROP_NO_OFFSET,
>     eStyleAnimType_None)

This looks like a bad merge that didn't compile; you need to put the "" back in.

r=dbaron with that

We should also think about a path to unprefixing this sooner rather than later...
Attachment #642961 - Flags: review?(dbaron) → review+
Comment on attachment 642962 [details] [diff] [review]
Changes to tests that depended on the old syntax

>diff --git a/layout/style/nsCSSPropList.h b/layout/style/nsCSSPropList.h

> CSS_PROP_TEXTRESET(
>     unicode-bidi,
>     unicode_bidi,
>     UnicodeBidi,
>     CSS_PROPERTY_PARSE_VALUE,
>+    "",
>     VARIANT_HK,
>     kUnicodeBidiKTable,
>     CSS_PROP_NO_OFFSET,
>     eStyleAnimType_None)
> CSS_PROP_USERINTERFACE(
>     -moz-user-focus,
>     user_focus,
>     CSS_PROP_DOMPROP_PREFIXED(UserFocus),

Aha.  This merge got in the wrong patch.  You should either put it in the other one or just squash the two patches together.

r=dbaron
Attachment #642962 - Flags: review?(dbaron) → review+
> We should also think about a path to unprefixing this sooner rather than later...

Authors shouldn't be using 'unicode-bidi' in general, so I don't see that as important. The values there mostly exist to support the relevant HTML features.
https://hg.mozilla.org/integration/mozilla-inbound/rev/6f224e55d00b
https://hg.mozilla.org/integration/mozilla-inbound/rev/d1f0011fdef2
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Keywords: dev-doc-needed
Summary: Implement unicode-bidi: isolate-override → Implement unicode-bidi: isolate-override (replacing ability to combine isolate and bidi-override keywords)
You need to log in before you can comment on or make changes to this bug.