-
Notifications
You must be signed in to change notification settings - Fork 93
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
Add view state for scroll prompts #43
Conversation
I'll need to do a more thorough test and code review, but my first impressions of the UX after checking this out locally are very positive! Well done! 🙌 |
@jessarcher mark as ready please when you're ready for me to take a look 👍 |
Hey @crazywhalecc! Sorry for the delay. Your scroll behaviour is perfect, but I've refactored the implementation to better fit my mental model. It also removes any state manipulation from the renderers so they can focus purely on rendering. Thanks for your PR and for getting this feature happening! |
I tried to resolve the issue #42 I created earlier, and this PR completed this feature: you can naturally scroll according to the options displayed on the current page, with minimal cost (perhaps).
As you know, I added a class to store view state, and changed the scroll behavior.
But I only make the virtical scroll naturally. The implementation of the horizontal direction is more complex, and the current plan in the horizontal direction feels tolerable.