Pinching the screen (especially with single hand) doesn't reliably zoom images and I have to try multiple times to get it to work

  • JustEnoughDucks@feddit.nl
    ·
    edit-2
    1 year ago

    Nope, I have the same issue, but I have narrowed down the exact behavior.

    If you put 2 fingers on the screen in a fullscreen image and slide them even the tiniest millimeter it will go into "pan mode" or so instead of zooming mode. You can no longer zoom until you release your fingers and place them without moving them at all before you pinch to zoom. (I have this happen much more often in practice while using the phone 1 handed on an unstable environment like a tram or train.

    Interestingly, you can pan and zoom at the same time if already zoomed in, but on a full image where panning doesn't do anything, it will lock both actions until you release your fingers.

    • moreeni@lemm.ee
      ·
      edit-2
      1 year ago

      Hm, I just tried it and it indeed does behave like you described, but only if you slide the fingers diagonally. If you slide them vertically you won't have this issue and it's the reason I didn't stumble upon it before

      Edit: wait, seems like this is not the case. It behaves weirdly, sometimes I can zoom just fine, sometimes sliding fingers won't zoom but do nothing.

    • Jerboa dev@programming.dev
      ·
      1 year ago

      It's hard to reproduce because it is not consistent but I see what you mean. Could you link the exact image that you managed to reproduce this. I'll see if I can report this to the image library that we use.

    • Jerboa dev@programming.dev
      ·
      1 year ago

      Could you test if you can still reproduce it in these builds?

      I have modified some code to isolate the problem.

      Release version of 46

      Depending on how you installed it might not allow you to install this release version

      Note replacing an existing install with this version, will probably delete your app info (it should tell you) use debug version instead if that is not wanted app-release_zoomable.zip

      Debug version of 46 app-debug_zoomable.zip

        • Jerboa dev@programming.dev
          ·
          1 year ago

          Can you also test if the problem is still fixed with this build and that the flick to exit works in the ImageViewer

          https://github.com/dessalines/jerboa/files/12618555/fix_zooming_jerboa_debug.zip

          • Lojcs@lemm.ee
            hexagon
            ·
            1 year ago

            It's not, as soon as the image moves vertically I can't zoom in anymore and it is easy to accidentally make it move before pinching. I think the image shouldn't respond to vertical swipes with two fingers at all and should have a small dead zone for single finger swipes. That's how WhatsApp seems to do it

            Also the black background takes too long to disappear after a flick, it could be nicer if it faded away