victory icon indicating copy to clipboard operation
victory copied to clipboard

VictoryZoomContainer: Zoom not working in mobile browsers

Open martin-probst opened this issue 10 months ago • 1 comments

Describe the bug While panning works fine, zoom does not work in mobile browsers (moving the finger in vertical direction or pinch gesture).

Victory version Victory 36.6.11. This is not about victory native, but about using standard web apps with victory in a mobile environment or as a PWA.

Code Sandbox link https://formidable.com/open-source/victory/guides/brush-and-zoom https://formidable.com/open-source/victory/docs/victory-zoom-container

To Reproduce Steps to reproduce the behavior: Open examples from the documentation like https://formidable.com/open-source/victory/guides/brush-and-zoom or https://formidable.com/open-source/victory/docs/victory-zoom-container in a mobile browser and try to zoom in. Neither moving vertically (scroll) nor pinching on the graph works to zoom into the graph like it does on a desktop browser.

Expected behavior It should be possible to somehow use zoom on mobile browsers as well. Pinching would be ideal, but handling vertical scrolling would work as well

Smartphone:

  • Device: Samsung S22+
  • OS: Android 13
  • Browser: Tested on both Chrome 117.0.5938.140 and Firefox 118.1.0

martin-probst avatar Oct 02 '23 07:10 martin-probst

Hi, we are transitioning all Native development to our new Victory Native XL library which is purpose built for supporting Native. Many of the design decisions we made in this version of Victory is not ideal for Native development.

carbonrobot avatar Nov 17 '23 20:11 carbonrobot