-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Auto scroll to nearest 'viewable' patch #26
Comments
There is a non-jQuery listener version that we should explore there |
scroll patch into center of view works: https://github.com/utk-se/CodeRibbon/pull/25/files#diff-0c38785aa45d91cacd92d3ff95aa20afR55 does not work for even number of patches, and if using start/end/nearest instead of center it often ends up with partially visible patches |
Where did you leave off on this? I know |
I see the commit, left a comment https://github.com/utk-se/CodeRibbon/pull/25/files#r259457220 |
HELP: How do you get more than 6 patches? Hahah. I wanted to tackle #18 but cannot even get to the horizontal scrolling on multiple patches. I messed around in settings and ended up seeing the scroll bar, but only after setting my min to 4 THEN back to 3. Weird... |
Yeah that's exactly how you add more... increase then decrease the number in settings lol |
Hahah alright, we can look into that as another issue |
#32 will make this easier |
Research required: https://github.com/lucafalasco/scroll-snap |
When user scrolls around, be able to load up the nearest set of patches automatically
The text was updated successfully, but these errors were encountered: