[css-scroll-snap] scroll-padding-block/inline and auto

[ewilligers]

The following properties have auto as their initial value, but their grammars only accept <length> values:

• scroll-padding-block
• scroll-padding-block-end
• scroll-padding-block-start
• scroll-padding-inline
• scroll-padding-inline-end
• scroll-padding-inline-start

https://drafts.csswg.org/css-scroll-snap-1/#padding-longhands-logical

[fantasai]

Good catch! Fixed, thanks so much for your attention to detail!!

[ewilligers]

Two shorthands still need fixing:

• scroll-padding-block
• scroll-padding-inline

[fantasai]

OK, I think it's good now?

[ewilligers]

The grammar for scroll-padding uses [ ].

I suspect scroll-padding-block and scroll-padding-inline need square brackets to, so they can accept auto 10px and 10px auto and auto auto.

[fantasai]

Man, even typoed the commit message and included a grammar error. I can't do anything right in this issue. T_T

Is the spec correct yet, @ewilligers ?

[ewilligers]

Looks good, thank you.