The spec was updated last month, and this polyfill still uses the old values.

ckrack / scrollsnap-polyfill

A polyfill for CSS scroll snapping

MIT License

217 stars 35 forks source link

The spec was updated last month, and this polyfill still uses the old values. #9

Open sdras opened 8 years ago

sdras commented 8 years ago

In this new version of the spec, https://drafts.csswg.org/css-snappoints/, only the scroll-snap-type has the same behavior- we now have new values like scroll-snap-padding.

ckrack commented 8 years ago

Thanks for reporting this. This will be part of a new release.

I don't have a milestone yet, as i am on vacation in Sri Lanka - it's literally too warm to work on it.

wjramos commented 8 years ago

Looking forward to this update

haraldox commented 8 years ago

@ckrack +1 would be great to have this! :)

ckrack commented 8 years ago

Currently short on time, but i'd be happy to merge pull requests if anyone wants to step ahead. I'd completely ditch the old spec, to keep things tidy.

wjramos commented 8 years ago

For clarity, the spec now supports the following properties:

scroll-snap-type
scroll-snap-align
scroll-snap-margin
scroll-snap-padding

Is there anything I'm missing? I'll take a crack at it

julesbou commented 7 years ago

Is there any progress on this issue? for sure it's a lot of work. Unfortunately we had to ditch this library as it wasn't working fine with the new spec.

kanzelm3 commented 6 years ago

+1 for this, new spec properties make scroll snapping much more concise and flexible IMO.