From 46cd218bb2325453c3a851b508c3b39e00160b1b Mon Sep 17 00:00:00 2001 From: Jia Date: Tue, 22 Aug 2017 16:33:20 +1000 Subject: [PATCH 1/2] Serialization description for position' --- css-typed-om/Overview.bs | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/css-typed-om/Overview.bs b/css-typed-om/Overview.bs index 4eddc34b..5ad932b1 100644 --- a/css-typed-om/Overview.bs +++ b/css-typed-om/Overview.bs @@ -300,7 +300,7 @@ The append(DOMString property, 3. If {{StylePropertyMap}}’s [=property model=] contains an entry for |property|, return the first value found in that entry. - + 4. Else, return `null`. @@ -2275,7 +2275,7 @@ objects, the {{CSSURLImageValue/url}} attribute contains the URL that references when called, perform the following steps: - 1. If the |url| passed into the constructor doesn't correctly + 1. If the |url| passed into the constructor doesn't correctly parse as a <>, throw a {{TypeError}} and exit this algorithm. 2. Else, return a new {{CSSURLImageValue}} with its {{CSSURLImageValue/url}} internal slot @@ -2879,7 +2879,7 @@ depending on the property they came from: 3. If 'border-image-slice' does not specify ''100%'' for all sides and omits the ''border-image-slice/fill'' keyword, serialize 'border-image-slice' and append it to |values|. 4. If 'border-image-width' does not specify ''1'' for all sides, - append "/ " (U+002F FORWARD SLASH followed by U+0020 SPACE) + append "/ " (U+002F FORWARD SLASH followed by U+0020 SPACE) to the result of serializing 'border-image-width' and append it to |values|. 5. If 'border-image-outset' does not specify ''0'' for all sides: 1. If the previous 'border-image-width' step did not append anything to |values|, @@ -2888,7 +2888,7 @@ depending on the property they came from: otherwise let |prefix| be "/ " (U+002F FORWARD SLASH followed by U+0020 SPACE) 2. Append |prefix| - to the result of serializing 'border-image-outset' + to the result of serializing 'border-image-outset' and append it to |values|. 6. If 'border-image-repeat' is not ''border-image-repeat/stretch'' in both axises, serialize 'border-image-repeat' and append it to |values|. @@ -2903,6 +2903,9 @@ depending on the property they came from: return "currentcolor". 2. Otherwise, return the result of serializing the <> value. +: 'position' +:: + 1. Return value of 'position'. Security Considerations {#security-considerations} From 3114f18b68ddad3d3b9716c680c9d273b89389aa Mon Sep 17 00:00:00 2001 From: Jia Date: Wed, 23 Aug 2017 11:52:34 +1000 Subject: [PATCH 2/2] Changed descriptions to address review comments' --- css-typed-om/Overview.bs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/css-typed-om/Overview.bs b/css-typed-om/Overview.bs index 6cbea6d1..89d862c6 100644 --- a/css-typed-om/Overview.bs +++ b/css-typed-om/Overview.bs @@ -2909,10 +2909,6 @@ depending on the property they came from: return "currentcolor". 2. Otherwise, return the result of serializing the <> value. -: 'position' -:: - 1. Return value of 'position'. - : 'left' :: 1. If the value is the ''auto'' keyword, @@ -2921,6 +2917,10 @@ depending on the property they came from: return the result of serializing the <> value. 3. Otherwise, return the result of serializing the <> value. +: 'position' +:: + 1. Return the serialization of 'position'. + : 'right' :: 1. If the value is the ''auto'' keyword,