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

RESOLVED FIXED in mozilla17

Status

()

Core
CSS Parsing and Computation
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: smontagu, Assigned: smontagu)

Tracking

({dev-doc-complete})

unspecified
mozilla17
dev-doc-complete
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Assignee)

Description

5 years ago
The latest version of CSS3 Writing Modes has a new value "isolate-override" for unicode-bidi, instead of the combination "bidi-override isolate".
(Assignee)

Comment 1

5 years ago
Created attachment 642961 [details] [diff] [review]
Patch
Assignee: nobody → smontagu
Attachment #642961 - Flags: review?(dbaron)
(Assignee)

Comment 2

5 years ago
Created attachment 642962 [details] [diff] [review]
Changes to tests that depended on the old syntax
Attachment #642962 - Flags: review?(dbaron)
(Assignee)

Comment 3

5 years ago
I'll move the hunk at https://bugzilla.mozilla.org/attachment.cgi?id=642962&action=diff#a/layout/style/nsCSSPropList.h_sec1 into attachment 642961 [details] [diff] [review]
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+

Comment 6

5 years ago
> 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.
(Assignee)

Comment 7

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/6f224e55d00b
https://hg.mozilla.org/integration/mozilla-inbound/rev/d1f0011fdef2
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17

Comment 8

5 years ago
https://hg.mozilla.org/mozilla-central/rev/6f224e55d00b
https://hg.mozilla.org/mozilla-central/rev/d1f0011fdef2
Keywords: dev-doc-needed
Summary: Implement unicode-bidi: isolate-override → Implement unicode-bidi: isolate-override (replacing ability to combine isolate and bidi-override keywords)
Updated:
https://developer.mozilla.org/en-US/docs/CSS/unicode-bidi
https://developer.mozilla.org/en-US/docs/Firefox_17_for_developers
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.