-
-
Notifications
You must be signed in to change notification settings - Fork 349
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
Not scrolling elements into view #26
Comments
Hi @5tormTrooper, Could you please give an eye to the code of this example? Then we can analyse which is the difference with your use case, maybe making changes on the sandbox. |
@5tormTrooper please let me know if you solved the Issue so we can close it. Otherwise, I'll ask you more information to reproduce the behavior you showed. Will close the Issue, feel free to re-open in case you need. |
@elrumordelaluz thanks for taking the time to demo this. After comparing the two, I released that there was a stray overflow rule on the app-body. After removing it the scrolling works fine. So it was a problem with my app and not the package. Sorry for the delayed response, other issues got promoted to top priority... Thanks again for looking into it! |
no problem.
hope the lib is useful for your app.
Il giorno 09 mar 2018, alle ore 11:32, Storm <notifications@github.com<mailto:notifications@github.com>> ha scritto:
@elrumordelaluz<https://github.com/elrumordelaluz> thanks for taking the time to demo this. After comparing the two, I released that there was a stray overflow rule on the app-body. After removing it the scrolling works fine. So it was a problem with my app and not the package.
Sorry for the delayed response, other issues got promoted to top priority...
Thanks again for looking into it!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#26 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AAv2uCA2PGa7758tGMByyvr_2jwLykIYks5tclpFgaJpZM4Rn-F0>.
|
Hi @5tormTrooper |
Found another example where it doesn't seem to work : |
reactour is not auto-scrolling to elements outside the viewport. My header and sidebar are fixed elements, and the main element contains the components, including those not being scrolled to.
My app has the following markup:
Any idea what could be preventing the scrolling?
The text was updated successfully, but these errors were encountered: