3liz / lizmap-web-client

Transfer a QGIS project on a server, Lizmap is providing the web interface to browse it
https://www.lizmap.com
Mozilla Public License 2.0
257 stars 143 forks source link

[Bug]: Incorrect work with the desktop touchscreen #3538

Open johnzet39 opened 1 year ago

johnzet39 commented 1 year ago

What is the bug?

I have a notebook with a touchscreen running Windows 11. If you use the touchscreen, and then start using the mouse or touchpad, it becomes unavailable to move the map and display popup information.

Steps to reproduce the issue

  1. Load map.
  2. Move map by touchscreen, get popup information by touchscreen.
  3. Try to move map by mouse or touchpad, try to get popup information.

if you don't have a touchscreen you can press F12 and toggle button "Toggle device toolbar". It will behave similarly.

Versions

Versions :

Check Lizmap plugin

QGIS server version, only if the section above doesn't mention the QGIS Server version

3.22

Operating system

Ubuntu

Browsers

Chrome, Microsoft Edge

Browsers version

110.0.5481.104

Relevant log output

No response

Gustry commented 1 year ago

Lizmap Web Client : 3.5.2

This version is old. Can you try to update ? (latest 3.5 or better latest 3.6)

johnzet39 commented 1 year ago

sorry, but unfortunately can't test other versions now. Can you reproduce this bug on later versions?

Gustry commented 1 year ago

Possible to use https://demo.lizmap.com

johnzet39 commented 1 year ago

Possible to use https://demo.lizmap.com

Thanks! Demo projects have the same behavior.

johnzet39 commented 1 year ago

I am adding a screenshot for completeness. Here is a variant with browser DevTools emulation of the mobile version. I remind you that initially we are talking about a Notebook with a touchscreen, in which the behavior is similar to this one without using Browser DevTools emulation. Untitled-3

nboisteault commented 1 year ago

Thanks @johnzet39 for your complete report. I've found the issue is in OpenLayers 2 that we still use for some parts in Lizmap. This issue will push us to migrate to OpenLayers 7 more quickly but there is still a lot of work.

nboisteault commented 1 year ago

Note to myself: I've tested to apply this commit we don't have in our OpenLayers.js but it did not fix the issue.