react-timeseries-charts icon indicating copy to clipboard operation
react-timeseries-charts copied to clipboard

Brush component is not working on mobile devices

Open tista3 opened this issue 7 years ago • 3 comments

You can test this on the cycling example: http://software.es.net/react-timeseries-charts/#/example/cycling?_k=h3dw4u Could it be handled like the amcharts do this? https://www.amcharts.com/kbase/getting-selection-chart-without-zooming/

tista3 avatar Jan 07 '18 10:01 tista3

Seems reasonable. There's quite a few issues on mobile devices unfortunately.

pjm17971 avatar Feb 06 '18 13:02 pjm17971

I've been hacking a similar component as brush using a slider (https://github.com/davidchin/react-input-range#readme). That one also has issues on mobile. In particular the onDrag (dragging the whole window/block) is not working on mobile.

Would love to see mobile support (or in our case tablet) for the brush.

I didn't check the ios/safari console for whats up, but I had a little trouble opening this link on my iphone: http://software.es.net/react-timeseries-charts/#/example/cycling?_k=h3dw4u

Thanks for the great library. Love the api.

acomito avatar Dec 20 '18 23:12 acomito

@acomito @tista3 unsure if you are still looking at using this. But I have made a PR that allows for the Brush to be used on mobile.

@pjm17971 Do you mind having a look?

jagribble avatar Jan 12 '20 02:01 jagribble