Skip to content
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

ScrollSnapMixin: an extension to achieve scroll-snap emulation with react-scroll #38

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

newsiberian
Copy link

Hello @fisshy,

please, take a look at this PR which brings new functionality to react-scroll. As said in title, my goal is to bring scroll-snap emulation to react-component to get the similar behavior like here or here (worked on wide screens only).

If you like it, please check it twice, because I'm just learning to programming;) Also, there is still some bugs, for example with touch events.

@fisshy
Copy link
Owner

fisshy commented Oct 15, 2015

I'm not really sure what this does or solves?

@newsiberian
Copy link
Author

Hi, @fisshy, have you seen an new example inside or links I that gave?

@newsiberian
Copy link
Author

What I was trying to achieve by these changes? I think that smooth scrolling can be used as is, but in conjunction with the snap-scrolling effect (which is optional) can be obtained good combination. Also please take a look at this post for more explanation.

@fisshy
Copy link
Owner

fisshy commented Oct 16, 2015

Alright, I'll go through the changes tonight and see how it works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants