48
submitted 1 year ago* (last edited 1 year ago) by Lojcs@lemm.ee to c/jerboa@lemmy.ml

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

you are viewing a single comment's thread
view the rest of the comments
[-] moreeni@lemm.ee 2 points 1 year ago

Are you sure this is not an issue with your phone? I can zoom just fine on Jerboa

[-] JustEnoughDucks@feddit.nl 12 points 1 year ago* (last edited 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.

[-] d_k_bo@feddit.de 2 points 1 year ago

I can confirm this exact behaviour.

[-] testAccount@programming.dev 1 points 1 year ago
[-] d_k_bo@feddit.de 1 points 1 year ago

OnePlus 5T with LineageOS 20/Android 13

[-] testAccount@programming.dev 1 points 1 year ago

Could you test if you can still reproduce it in these builds (I can not test it myself as I can not seem to reproduce it)

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

[-] d_k_bo@feddit.de 1 points 1 year ago

This build seems to fix this issue! Sorry, that I didn't test it sooner.

[-] testAccount@programming.dev 1 points 1 year ago

np

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

[-] d_k_bo@feddit.de 1 points 1 year ago

It's pretty much back to the behavior explained above. If you slightly move your fingers before zooming in, you are stuck at flicking the image away and can't zoom.

[-] testAccount@programming.dev 2 points 1 year ago

Alright we will be dropping this feature until we make a proper swipe to exit

[-] testAccount@programming.dev 1 points 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.

[-] moreeni@lemm.ee 1 points 1 year ago* (last edited 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.

[-] testAccount@programming.dev 1 points 1 year ago

Which device do you use?

[-] testAccount@programming.dev 1 points 1 year ago

Also can you do this too:

Could you test if you can still reproduce it in these builds (I can not test it myself as I can not seem to reproduce it)

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

[-] testAccount@programming.dev 1 points 1 year ago

Which device do you use? It might be device specific. Hard to reproduce for me

[-] testAccount@programming.dev 1 points 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

[-] Lojcs@lemm.ee 1 points 1 year ago

This does fix the issue for me

[-] testAccount@programming.dev 2 points 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 1 points 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

[-] JustEnoughDucks@feddit.nl 1 points 1 year ago* (last edited 1 year ago)

There is a release 46 on f-droid. This does not fix the issue (tested on a few photos including this https://i.redd.it/mm6682q8tgmb1.jpg and this https://i.imgur.com/XRKZAhR.jpeg)

Perhaps it is an issue with embedding from reddit images?

If you still want me to, I cab probably install the debug version and attempt further debugging.

[-] testAccount@programming.dev 3 points 1 year ago

The release on Fdroid does not contain these experimental changes.

Only the above builds

this post was submitted on 31 Aug 2023
48 points (98.0% liked)

Jerboa

10133 readers
24 users here now

Jerboa is a native-android client for Lemmy, built using the native android framework, Jetpack Compose.

Warning: You can submit issues, but between Lemmy and lemmy-ui, I probably won't have too much time to work on them. Learn jetpack compose like I did if you want to help make this app better.

Built With

Features

Installation / Releases

Support / Donate

Jerboa is made by Lemmy's developers, and is free, open-source software, meaning no advertising, monetizing, or venture capital, ever. Your donations directly support full-time development of the project.

Crypto

Contact

founded 2 years ago
MODERATORS