Re: [csswg-drafts] [css-backgrounds-4] Added box-shadow-* longhand properties (#6083)

It's hard to review this properly because the entire text from backgrounds-3 is brought in and then edited, so it's hard to know what the delta is. The PR as it currently stands includes stuff about drawing shadows, which is not relevant to breaking down the syntax into longhands.

Wrt to the longhands, some thoughts, not very organized:
- `box-shadow-offset` is not consistent naming with any other property that takes a horizontal and vertical offset. 
- We should have separate horizontal and vertical longhands. Not sure if these should be physical (like `background-position`) or logical. I defer to @fantasai on that.
- If we name the offset property `box-shadow-position`, we should name the one for `inset` differently. Perhaps `box-shadow-placement`?
- Yes, CSS is internally inconsistent here. We have conflicting precedents for `*-position` properties: `list-style-position` to follow for `inset | outset` and things like `background-position`. 
- OTOH `background-position` does not specify *offsets* from an initial position, it specifies a whole new position. So maybe that is not a good precedent to follow. What else in CSS specifies offsets? There is `top` and `left`, `translate`, what else?

-- 
GitHub Notification of comment by LeaVerou
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/pull/6083#issuecomment-1232734049 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Wednesday, 31 August 2022 10:03:58 UTC