From d412088b363cb15d6447de4fd022319f67c47b9a Mon Sep 17 00:00:00 2001 From: Sebastian Zartner Date: Wed, 19 Jul 2023 20:14:07 +0200 Subject: [PATCH 1/6] [css-borders-4] Fixed and added definitions for box-shadow-offset: none --- css-borders-4/Overview.bs | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/css-borders-4/Overview.bs b/css-borders-4/Overview.bs index 82e740bc7d3..f45f7ce1b4b 100644 --- a/css-borders-4/Overview.bs +++ b/css-borders-4/Overview.bs @@ -617,16 +617,24 @@ Initial: none Applies to: all elements Inherited: no Percentages: N/A -Computed value: either 'none' or a list, - each item a pair of offsets (horizontal and vertical) from the element‘s box +Computed value: list, each item either 'none' or a pair of offsets + (horizontal and vertical) from the element‘s box Animation type: by computed value

The 'box-shadow-offset' property defines one or more drop shadow offsets. -The property accepts a comma-separated list of horizontal and vertical offset pairs, +The property accepts a comma-separated list. +Each item in that list can either be the ''box-shadow-offset/none'' value, +which indicates no shadow, or a pair of horizontal and vertical offsets, where both values are described as <> values.

+
none +
+ No box shadow will be created. + Any other box shadow properties specified for this box shadow have no + effect. +
1st <>
Specifies the horizontal offset of the shadow. From b0da2bd23450171731b13dc87135f9ffc7c0f844 Mon Sep 17 00:00:00 2001 From: Sebastian Zartner Date: Wed, 19 Jul 2023 20:35:05 +0200 Subject: [PATCH 2/6] [css-borders-4] Fixed 'box-shadow' shorthand definition --- css-borders-4/Overview.bs | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/css-borders-4/Overview.bs b/css-borders-4/Overview.bs index f45f7ce1b4b..1e38bdfe664 100644 --- a/css-borders-4/Overview.bs +++ b/css-borders-4/Overview.bs @@ -749,14 +749,12 @@ Initial: none Applies to: all elements Inherited: no Percentages: N/A -Computed value: either the keyword ''box-shadow-offset/none'' or - a list, each item consisting of four absolute lengths - plus a computed color and optionally also a ''box-shadow-position/inset'' keyword +Computed value: see individual properties Animation type: by computed value, - treating ''box-shadow-offset/none'' as a zero-item list - and appending blank shadows (''transparent 0 0 0 0'') - with a corresponding ''box-shadow-position/inset'' keyword as needed - to match the longer list + letting ''box-shadow-offset/none'' create a blank shadow + (''transparent 0 0 0 0'') when interpolated with non-'none' values + and appending blank shadows with a corresponding + ''box-shadow-position/inset'' keyword as needed to match the longer list if the shorter list is otherwise compatible with the longer one From 668a239cac0e55caf017972da042d546eaf30ea6 Mon Sep 17 00:00:00 2001 From: Sebastian Zartner Date: Wed, 26 Jul 2023 12:59:44 +0200 Subject: [PATCH 3/6] [css-borders-4] Moved animation type to box-shadow-offset and fixed color value --- css-borders-4/Overview.bs | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/css-borders-4/Overview.bs b/css-borders-4/Overview.bs index 1e38bdfe664..1a84bb66bc9 100644 --- a/css-borders-4/Overview.bs +++ b/css-borders-4/Overview.bs @@ -619,7 +619,12 @@ Inherited: no Percentages: N/A Computed value: list, each item either 'none' or a pair of offsets (horizontal and vertical) from the element‘s box -Animation type: by computed value +Animation type: by computed value, + letting ''box-shadow-offset/none'' create a blank shadow + (''transparent 0 0 0 0'') when interpolated with non-'none' values + and appending blank shadows with a corresponding + ''box-shadow-position/inset'' keyword as needed to match the longer list + if the shorter list is otherwise compatible with the longer one

The 'box-shadow-offset' property defines one or more drop shadow offsets. @@ -750,12 +755,7 @@ Applies to: all elements Inherited: no Percentages: N/A Computed value: see individual properties -Animation type: by computed value, - letting ''box-shadow-offset/none'' create a blank shadow - (''transparent 0 0 0 0'') when interpolated with non-'none' values - and appending blank shadows with a corresponding - ''box-shadow-position/inset'' keyword as needed to match the longer list - if the shorter list is otherwise compatible with the longer one +Animation type: see individual properties

The 'box-shadow' property attaches one or more drop-shadows to the box. From f90d2a25bb21b91fa8e892c65d73c3c2185b5a8f Mon Sep 17 00:00:00 2001 From: Sebastian Zartner Date: Thu, 27 Jul 2023 22:51:40 +0200 Subject: [PATCH 4/6] [css-borders-4] Fixed animation type of `box-shadow-offset` and interaction with `box-shadow-color` --- css-borders-4/Overview.bs | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/css-borders-4/Overview.bs b/css-borders-4/Overview.bs index 1a84bb66bc9..0291c8e2805 100644 --- a/css-borders-4/Overview.bs +++ b/css-borders-4/Overview.bs @@ -620,11 +620,8 @@ Percentages: N/A Computed value: list, each item either 'none' or a pair of offsets (horizontal and vertical) from the element‘s box Animation type: by computed value, - letting ''box-shadow-offset/none'' create a blank shadow - (''transparent 0 0 0 0'') when interpolated with non-'none' values - and appending blank shadows with a corresponding - ''box-shadow-position/inset'' keyword as needed to match the longer list - if the shorter list is otherwise compatible with the longer one + treating ''box-shadow-offset/none'' as ''0 0'' + when interpolated with non-'none' values.

The 'box-shadow-offset' property defines one or more drop shadow offsets. @@ -759,13 +756,14 @@ Animation type: see individual properties

The 'box-shadow' property attaches one or more drop-shadows to the box. - The property accepts either the ''box-shadow-offset/none'' value, which indicates no shadows, - or a comma-separated list of shadows, ordered front to back. + The property accepts a comma-separated list of shadows, + ordered front to back.

Each shadow is given as a <>, outlining the 'box-shadow-offset', and optional values for the 'box-shadow-blur', 'box-shadow-spread', 'box-shadow-color', and 'box-shadow-position'. - Omitted lengths are 0; omitted colors default to the ''currentcolor'' value. + If the offset is ''box-shadow-offset/none'', + ''box-shadow-color'' is treated as ''transparent''. 

   <> = <<'box-shadow-color'>>? && [ <<'box-shadow-offset'>> [ <<'box-shadow-blur'>> <<'box-shadow-spread'>>? ]? ] && <<'box-shadow-position'>>?
From 532f74c12fb7e6e4d9dcb23d29fafb0a0e864ffe Mon Sep 17 00:00:00 2001 From: Sebastian Zartner Date: Thu, 27 Jul 2023 22:57:20 +0200 Subject: [PATCH 5/6] [css-borders-4] Clarified the `none` value of `box-shadow-offset` --- css-borders-4/Overview.bs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/css-borders-4/Overview.bs b/css-borders-4/Overview.bs index 0291c8e2805..bac23500f89 100644 --- a/css-borders-4/Overview.bs +++ b/css-borders-4/Overview.bs @@ -634,8 +634,8 @@ where both values are described as <> values.
none
No box shadow will be created. - Any other box shadow properties specified for this box shadow have no - effect. + Any other box shadow properties specified for that shadow in the list + of box shadows have no effect.
1st <>
From 8178637147d0f43693c9d1d2e40150cc4b3d7832 Mon Sep 17 00:00:00 2001 From: Sebastian Zartner Date: Mon, 31 Jul 2023 13:05:36 +0200 Subject: [PATCH 6/6] [css-borders-4] Improved phrasing of `box-shadow-offset: none` and expansion of `box-shadow` shorthand --- css-borders-4/Overview.bs | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/css-borders-4/Overview.bs b/css-borders-4/Overview.bs index bac23500f89..6842caef17c 100644 --- a/css-borders-4/Overview.bs +++ b/css-borders-4/Overview.bs @@ -633,9 +633,8 @@ where both values are described as <> values.
none
- No box shadow will be created. - Any other box shadow properties specified for that shadow in the list - of box shadows have no effect. + The shadow will not be rendered. + The values of other box shadow properties corresponding to this shadow have no effect.
1st <>
@@ -762,8 +761,10 @@ Animation type: see individual properties

Each shadow is given as a <>, outlining the 'box-shadow-offset', and optional values for the 'box-shadow-blur', 'box-shadow-spread', 'box-shadow-color', and 'box-shadow-position'. - If the offset is ''box-shadow-offset/none'', - ''box-shadow-color'' is treated as ''transparent''. + Omitted lengths are '0'; + omitted colors default to ''transparent'' when the specified offset is ''box-shadow-offset/none'' + and to ''currentcolor'' otherwise. + 

   <> = <<'box-shadow-color'>>? && [ <<'box-shadow-offset'>> [ <<'box-shadow-blur'>> <<'box-shadow-spread'>>? ]? ] && <<'box-shadow-position'>>?