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

Scrolling problem for the turbo finger #102

Open
pdattx opened this issue May 29, 2017 · 3 comments
Open

Scrolling problem for the turbo finger #102

pdattx opened this issue May 29, 2017 · 3 comments

Comments

@pdattx
Copy link
Contributor

pdattx commented May 29, 2017

If you try to scroll a little bit faster then the average (especially on a smartphone) you have such big jumps through the bible and a strange effect (that even if you leave the finger than on the touchscreen and swipe around the jumps goes further). You can test it with the biblewebapp.com as well.
But the fix seems to be easy: Go to the scroller.js in the windows-folder and comment out the lines ~240 and ~257 : "wrapper.find('.section:first').remove();" and "wrapper.find('.section:last').remove();"

I know that is not the solution, but it helps to have a more direct navigation-control to the application.

Maybe there is another way to remove an exact section without freaking out with the navigation!?

@johndyer
Copy link
Member

johndyer commented Jun 1, 2017

@pdattx Thanks for this! A few months ago the desktop version of Chrome made some changes, so I changed the the way the system inserted a chapter above the current viewport (lines ~333-363). But I too noticed that now the iPhone browser is getting pretty jumpy again. I know iOS is changing some scrolling behavior to work with Google's AMP and I'm wondering if that's affecting the calculation.

When I tested your idea of removing the code that tries to limit the number of chapters loaded (lines ~203-260), it seemed to help in scrolling downward. But I'm still getting a bit of jumpiness when I scroll upward on an iPhone. Did it solve issues for you in scrolling upward? Are you on iOS or Andriod?

@pdattx
Copy link
Contributor Author

pdattx commented Jun 1, 2017

I'm on iOS, but I must reset my changes in scroller.js. It helps in both directions, yes - that was what I realized - but there were some other effects (it think it was the jump to another chapter or so) that I have to reactivate this lines again.
PS: I've taken the last chrome-update as well, but the effect is still there.

@pdattx
Copy link
Contributor Author

pdattx commented Jun 5, 2017

It's also possible to reproduce it on a PC. You must only be fast and if you resize the browser it seems to work faster wrong.

PS: could it be, that in such a case the jump to another chapter is not possible?

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

No branches or pull requests

2 participants