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

Activity

1. ewilligers

   added

   css-scroll-snap-1Current WorkCurrent Work

   on Oct 2, 2018
2. fantasai

   added a commit that references this issue on Oct 3, 2018

   [css-scroll-snap] Add missing auto value to scroll-padding longhands. #…

   ...

   abc1853
3. fantasai commented on Oct 3, 2018

   

   fantasai

   on Oct 3, 2018

   Collaborator

   More actions

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

   added

   Closed Accepted as Obvious BugfixFixed an obvious error with the obvious solution.Fixed an obvious error with the obvious solution.

   on Oct 3, 2018
5. fantasai

   closed this as completedon Oct 3, 2018
6. ewilligers commented on Oct 4, 2018

   

   ewilligers

   on Oct 4, 2018

   ContributorAuthor

   More actions

   Two shorthands still need fixing:

   • scroll-padding-block
   • scroll-padding-inline
7. ewilligers

   reopened this on Oct 4, 2018
8. fantasai

   added a commit that references this issue on Oct 5, 2018

   [css-scroll-snap] Fix missing auto value in scroll-padding-inline/blo…

   ...

   5dfc463
9. fantasai commented on Oct 5, 2018

   

   fantasai

   on Oct 5, 2018

   Collaborator

   More actions

   OK, I think it's good now?
10. ewilligers commented on Oct 5, 2018

    

    ewilligers

    on Oct 5, 2018

    ContributorAuthor

    More actions

    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.

    👍React with 👍2Reacted by jonjohnjohnson and Sebastian Zartner
11. fantasai

    added a commit that references this issue on Oct 8, 2018

    [css-scroll-snap-1] Fix scroll-padding-inline/block grammar ad hope e…

    ...

    622c8f1
12. fantasai commented on Oct 8, 2018

    

    fantasai

    on Oct 8, 2018

    Last edited by fantasai

    Collaborator

    More actions

    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 ?
13. ewilligers commented on Oct 9, 2018

    

    ewilligers

    on Oct 9, 2018

    ContributorAuthor

    More actions

    Looks good, thank you.
14. ewilligers

    closed this as completedon Oct 9, 2018
15. chromium-wpt-export-bot

    added a commit that references this issue on Oct 25, 2018

    Update scroll-padding initial values in wpt tests.

    ...

    4c95390
16. aarongable

    added a commit that references this issue on Oct 25, 2018

    Update scroll-padding initial values in wpt tests.

    ...

    8689480
17. chromium-wpt-export-bot

    added 2 commits that reference this issue on Oct 25, 2018

    Update scroll-padding initial values in wpt tests.

    ...

    be749ce

    Update scroll-padding initial values in wpt tests.

    ...

    f5832cc
18. moz-v2v-gh

    added a commit that references this issue on Nov 10, 2018

    Bug 1502231 [wpt PR 13726] - Update scroll-padding initial values in …

    ...

    fa7c233
19. jyc

    added a commit that references this issue on Nov 11, 2018

    Bug 1502231 [wpt PR 13726] - Update scroll-padding initial values in …

    ...

    492e09d
20. gecko-dev-updater

    added 3 commits that reference this issue on Oct 3, 2019

    Bug 1502231 [wpt PR 13726] - Update scroll-padding initial values in …

    ...

    0968b24

    Bug 1502231 [wpt PR 13726] - Update scroll-padding initial values in …

    ...

    437eba7

    Bug 1502231 [wpt PR 13726] - Update scroll-padding initial values in …

    ...

    98dd923
21. jwidar

    added a commit that references this issue on Sep 16, 2025

    Bug 1502231 [wpt PR 13726] - Update scroll-padding initial values in …

    ...

    0d453d0