-
Notifications
You must be signed in to change notification settings - Fork 657
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[css-backgrounds-4] Allow repeating none
in box-shadow-offset
#8567
Comments
I don't personally mind the stacking as a reader. However, I note that the Value Definition Syntax does not envision that stacking for now, and some parsers do not support this syntax. I reported this as an issue to CSSTree for instance, see csstree/csstree#249 |
Ah, we should allow that stacking pattern probably. (Probably we should just allow all of them; I'm not sure why we're being so precious about what stacks are allowed when they can all be expressed with brackets anyway.) |
Ah no, we have that for a reason actually - given that |
I understand that the use of People are saying that implementing coordinating list value is hard in the transcript for #7164. I am not sure I identify the exact nature of the complexity but it could perhaps be decreased a bit by consistently allowing And why prevent the author from using Below is a summary of the restrictions on
In my opinion, |
I'm not sure what's unclear about it. |
Right, sorry, I thought the application order was something you did not want to define explicitly, but it already is...
... and this has nothing to do with the reason why |
@cdoublev provided good reasoning for The discussion about stacking multipliers together is orthogonal to that issue (and with the change irrelevant for that specific case). So we should create a new issue for that. Also, Sebastian |
Reopening because if
And also I don't see the definition of what |
Yes, I realized that too. Considering that |
Then I'm confused, what's the point of having |
Nevermind. I think it should cancel the shadow, as defined in CSS Backgrounds 3, similarly as in
But it seems even weirder that |
That is because the offset is the only thing required in the Sebastian |
none, none
is currently valid inbox-shadow
but not inbox-shadow-offset
.animation
,transition
,scroll-timeline
,view-timeline
, accept declaringnone
with other values and repeating it in a coordinating value list. In #8440, I suggested restrictingnone
in these properties, before considering some (contrived) use cases.Nit: stacking
{1,2}
and#
does not seem invalid but some spec readers may think[ <length>{1,2} ]#
is clearer.The text was updated successfully, but these errors were encountered: