suii2210 commented on issue #21261: URL: https://github.com/apache/echarts/issues/21261#issuecomment-3598603650
Hey, I’d like to pick this issue up. I tested the example on both a real phone and DevTools. On an actual device the brush works fine, but in DevTools mobile mode the drag sometimes triggers page scrolling instead of moving the brush. My plan is pretty simple: When the user starts dragging the brush, I’ll temporarily set touch-action: none on the chart’s DOM so the browser doesn’t try to scroll the page. When the drag ends, I’ll reset it back. And for environments that don’t respect touch-action, I’ll add a non-passive touchmove/pointermove listener with preventDefault() only while the brush is active. This keeps normal scrolling intact and only blocks it during the actual brush drag, so it’s safe. If this approach looks good, I’ll start working on the PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
