andreynovikov / trekarta

Simple, responsive map for your trek. You decide how to journey today, not the application!
https://trekarta.info
GNU General Public License v3.0
176 stars 25 forks source link

Map becomes unusable when zooming and changing the orientation at the same time #184

Open billie80 opened 2 years ago

billie80 commented 2 years ago

This is really hard to explain. Under some conditions, I think when zooming and changing the orientation at the same time no map is displayed anymore. Zoom level is at 10000 km and only a blank view is displayed. The only way to make the App usable again is deleting the App data.

Here a screenshot of who the App looks after it happened: bugreport-axolotl-QSV1 210329 032-2022-04-19-19-49-24

I have an Android bug report ready which I can share privately.

andreynovikov commented 2 years ago

Have never met this. Will think how to fix that.

billie80 commented 2 years ago

Thanks for looking at it. Actually I experience it rather often. If you need anything which could help solving this just ask.

andreynovikov commented 2 years ago

Please enable zoom buttons in Settings. Next time it will happen will zoom button let you zoom in?

billie80 commented 2 years ago

Tried this now. The zoom buttons don't allow me to zoom in if it happens.

billie80 commented 2 years ago

I think I can reliably reproduce it by switching between portrait and landscape orientation while at the same time zooming in and out! Please see the screencast below. I have enabled taps so the zoom actions are visible.

https://user-images.githubusercontent.com/9938306/165391077-e4772f90-4cee-4c80-b583-28fe4fd8f2b2.mp4

nettybun commented 2 years ago

@billie80 What version? I installed 2022.04 today and can't reproduce it. It seems the default Android 12 screenrecorder doesn't show finger location unfortunately, but trust that I'm trying my best to zoom and orient at the same time.

In my case it seems that Trekarta doesn't perform the reorientation until after I've released my fingers. If I keep them on the screen it'll lock screen rotation despite me physically rotating the phone. It only rotates with no fingers on it. Maybe this is an Android 12 thing?

https://user-images.githubusercontent.com/44614862/165435971-0f34048c-0bb7-47fa-a191-ce8fc97ef6f8.mp4

nettybun commented 2 years ago

Tested with 4 different apps - it seems it's Android that's blocking the rotation until fingers are released from the screen. I guess this means the bug isn't possible to reproduce for this OS version :/

billie80 commented 2 years ago

Still on 2021.04. Waiting for F-Droid to pick up the new version. It builds from source and checks for new tags on GitHub but it seems the new version was released to the Play Store before there was a GitHub tag. It will be in the next build cycle. If it successfully builds, maybe I can test it in a few days. (Edit: The build failed because they need to update the build recipe manually if an App requires Java 11. So waiting a few more days until this happens. If there is really something in there I could also switch to the Play Store build butbI don't think so.). @andreynovikov Any chance to create the tag earlier?

I am on Android 10. I enabled the finger taps in the developer options. Maybe this is possible on Android 12 as well. It is not perfectly visible in my video but I think rotating an zooming do not happen simultaneously. It just happens in a quick sequence. As you say if I zoom and keep the fingers on the screen, orientation does not change when I rotate the phone. This only happens after the fingers are released.

andreynovikov commented 2 years ago

Yes, I forgot to tag last release, figured it out several days after.

billie80 commented 2 years ago

Just want to let you know that it still happens with version 2022.04.

nettybun commented 2 years ago

@billie80 have you been able to reproduce it on more than one Android device? I've tried on 3 devices (Pixel 5a, Essential PH1, and LG K20) and cannot reproduce yet :/

tomate44 commented 2 years ago

Hello, This bug happened to me twice also. I am using a Gigaset GS290, running /e/OS, based on Android 10. Trekarta version is 2021.04, from F-droid. Zoom buttons didn't fix the bug. BTW, thanks a lot for this app, I really love it.

billie80 commented 2 years ago

@heyheyhello unfortunately I don't have other devices to test at the moment. @tomate44 thanks for confirming this on another device. I also run Android 10 without GMS but don't think this is the reason. I have access to the same device as mine with GMS. I can try there to make sure.