-
Notifications
You must be signed in to change notification settings - Fork 109
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
Some scrolling oddities when changing views #73
Comments
Is this #38 ? |
Nope, it is different. This may affect it though. In this case the steps to reproduce the behavior are as seen below. you need a long playlist and a short playlist.
What is expected is the queue being at the top or last location it was at. However you end up at the scroll value of the short playlist, somewhere in the middle.
What is expected is that the queue will be scrolled to the top or where you last left it. However since the playlist was longer than the queue, the page is scrolled all the way to the bottom. This also happens if you scroll to the bottom of the queue, and then switch to the short playlist. |
Ahh ok I follow you now. Yeh, let's fix it. |
So there's a proposed fix in #93 and there's a similar idea at http://stackoverflow.com/questions/11216392/how-to-handle-scroll-position-on-hashchange-in-backbone-js-application. The latter might be a little neater but I'm open to whatever works. |
Looking really forward for this fix! At the moment it's really annoying to browse through the library :( |
Just a hint: Mopidy-Mobild can remember the scroll-position: https://github.com/tkem/mopidy-mobile |
I did notice, along the lines of #71 and scrolling the view that the view isn't scrolled on view changes.
The scrollbar appears to remain at the same length value, which if the new view is shorter will result in arriving at the bottom of the page and if the new view was actually quite a bit longer it will be at some point in the middle. I think it would be nice to scroll to the top on new page view loads as a courtesy.
The text was updated successfully, but these errors were encountered: