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

Created on 2 Oct 2018  Â·  6Comments  Â·  Source: w3c/csswg-drafts

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

Closed Accepted as Obvious Bugfix css-scroll-snap-1

Most helpful comment

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.

All 6 comments

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

Two shorthands still need fixing:

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

OK, I think it's good now?

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.

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 ?

Looks good, thank you.

Was this page helpful?
0 / 5 - 0 ratings