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

slider broken on mobile #62

Open
pbellec opened this issue Mar 18, 2020 · 3 comments
Open

slider broken on mobile #62

pbellec opened this issue Mar 18, 2020 · 3 comments
Labels
bug Something isn't working plotly Requires skills with plotly express

Comments

@pbellec
Copy link

pbellec commented Mar 18, 2020

This is a very useful tool, better done than what I have seen so far.
Congrats to the team!

The experience on mobile is not optimal though, as the controls are really small.
But it is workable.
The only real issue I have noticed is the zoom tool (gray shaded area).
It does not seem to respond to touch.
But the zoom in on the curve does work.
This actually makes the dashboard unusable as soon as you start zooming in.

@pbellec pbellec added the bug Something isn't working label Mar 18, 2020
@GaelVaroquaux GaelVaroquaux added the dash Requires Dash skills label Mar 18, 2020
@GaelVaroquaux
Copy link
Contributor

Cc @nicolaskruchten : any advice for mobile?

@nicolaskruchten
Copy link
Collaborator

Hmmm yeah the rangeslider might not be mobile-enabled in Plotly.js. Not much I can offer by way of workarounds unfortunately :(

Basically users can double-tap the main plotting area to zoom all the way out and then tap-drag to zoom back in to the desired area.

On a design level, I might suggest removing the rangeslider?

@GaelVaroquaux
Copy link
Contributor

GaelVaroquaux commented Mar 18, 2020 via email

@emmanuelle emmanuelle added plotly Requires skills with plotly express and removed dash Requires Dash skills labels Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working plotly Requires skills with plotly express
Projects
None yet
Development

No branches or pull requests

4 participants