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

The charts consume all scroll events #92

Open
gregkorossy opened this issue Feb 21, 2024 · 2 comments
Open

The charts consume all scroll events #92

gregkorossy opened this issue Feb 21, 2024 · 2 comments
Labels
bug Something isn't working open Anyone interested in working on this please let me know.

Comments

@gregkorossy
Copy link

gregkorossy commented Feb 21, 2024

If the chart takes up a significant amount of a scrolling container, the user cannot scroll down because even though the chart only cares about the horizontal dragging, it consumes the vertical gestures as well.
To test it, open the app, navigate to the multi-line chart screen, rotate your device to landscape and try scrolling down naturally (so basically anywhere in the container).

@dautovicharis dautovicharis added bug Something isn't working open Anyone interested in working on this please let me know. labels Feb 21, 2024
@carstenhag
Copy link

I noticed the same while scrolling there (phone in portrait mode)

@dautovicharis
Copy link
Owner

dautovicharis commented Feb 21, 2024

Thanks @gregkorossy and @carstenhag. I will plan this for the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working open Anyone interested in working on this please let me know.
Projects
None yet
Development

No branches or pull requests

3 participants