bk138 / multivnc

MultiVNC is a cross-platform Multicast-enabled VNC viewer based on LibVNCClient. The desktop client runs on Unix, Mac OS X and Windows. There also is an Android client.
GNU General Public License v3.0
464 stars 66 forks source link

Zooming broken on Android 12 #183

Closed MrPeteH closed 2 years ago

MrPeteH commented 2 years ago

If you'd like to put out an incentive for fixing this bug, you can do so at https://issuehunt.io/r/bk138/multivnc?tab=idle

Is your bug report about the Desktop Multivnc or the Mobile MultiVNC?

Which MultiVNC version are you using? 2.0.5

Which server are you connecting to?

Maybe important:

Describe the bug MultiVNC is unusable due to large white rectangles that overlay the screen.

If I had to guess,

To Reproduce

For the Mobile Version (please complete the following information):

Additional context 2022-02-08 14 04 04 2022-02-08 14 02 34

bk138 commented 2 years ago

Thanks for the report!

Few questions:

MrPeteH commented 2 years ago

Does not happen on any other viewer I've tried (I've tried several, including freebVNC)

I'll work on the latter test :)

bk138 commented 2 years ago

@MrPeteH nevermind, I can repro this on a Galaxy S10 with an example server that's set to 3840 x 1197. Does not happen with AVNC though.

bk138 commented 2 years ago

Might be limitations of glTexImage2D, but AVNC does it similarly... :thinking:

MrPeteH commented 2 years ago

S10e does not have the issue. Viewer 1080x2020 at 480 dpi.

So far more a resolution than dpi thing...

bk138 commented 2 years ago

This actually happens with every server resolution on my device now. I can state it worked before, so it's likely the recent update to Android 12 that changed something...

bk138 commented 2 years ago

If I remove VncCanvas.scrollToAbsolute(), it works on Android 12 again. Will test with other devices the next days...

bk138 commented 2 years ago

test results

v2.0.5

scrollToAbsolute() removed

MrPeteH commented 2 years ago

Yep. Fixed. Now I can see the next issue more clearly 🤠