Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] when zooming fast back and forth the map gets to a state that is stuck #563

Open
ronlv10 opened this issue Oct 5, 2024 · 8 comments
Labels
bug Something isn't working

Comments

@ronlv10
Copy link

ronlv10 commented Oct 5, 2024

Description

When using a phone and zooming fast back and forth (out in out) with a map with advanced markers + clustering, sometimes the map gets to a state that it's stuck (and "going crazy")

after it happens a user can't do anything on the map and sometimes it's stuck in a way that reloads the tab doesn't work.

See:

WhatsApp.Video.2024-10-05.at.09.36.41.mp4

P.S It also happened in my implementation of clustering on a map with an app that I have built.

Steps to Reproduce

not an easy one. I used the clustering demo : https://visgl.github.io/react-google-maps/examples/custom-marker-clustering

you should use your phone and zoom in and out aggressively

Environment

  • Library version: 1.3.0
  • Google maps version: weekly
  • Browser and Version: chrome
  • OS: android

Logs

there is an warning when it happens: 

Cannot remove a WebglOverlay that has not been set to a map
@ronlv10 ronlv10 added the bug Something isn't working label Oct 5, 2024
@ronlv10
Copy link
Author

ronlv10 commented Oct 5, 2024

the error is the same as #419 but in this bug it says that there is no impact on the app. Here the bug is critical.

@ronlv10
Copy link
Author

ronlv10 commented Nov 6, 2024

hey, @usefulthink any comment?
This bug impacts our app severely

@usefulthink
Copy link
Collaborator

As far as I know, the issue in #419 has been resolved upstream in the maps API itself. Are you still seeing that warning-message?

@usefulthink
Copy link
Collaborator

And you're saying it is severely affecting your app, can you elaborate on that a bit? I was under the impression that the original issue only happens occasionally when aggressively zooming around the screen, which isn't something I'd expect users to do.

@ronlv10
Copy link
Author

ronlv10 commented Nov 10, 2024

And you're saying it is severely affecting your app, can you elaborate on that a bit? I was under the impression that the original issue only happens occasionally when aggressively zooming around the screen, which isn't something I'd expect users to do.

It happened to me multiple times while doing usability session with users. It doesn't happen only when zooming aggressively but it was my way to reproduce it faster.
Here are two videos that I made now, zooming slowly back and forth until it happens: one from the demo clustering and one from my app. (please note it happens only with mobile zoom (two fingers)

Demo:

screen-20241110-130304.1.mp4

My app:

screen-20241110-132322.online-video-cutter.com.1.1.mp4

As far as I know, the issue in #419 has been resolved upstream in the maps API itself. Are you still seeing that warning-message?

After upgrade the package version I didn't see the warning, I did saw now this error:

InvalidValueError: setCenter: not a LatLng or LatLngLiteral with finite coordinates: in property lng: -Infinity is not an accepted value

PS
After reproducing it millions of times, I suspect that it happens when the user zooms out and keeps the fingers on the screen (and probably touching the markers)

@usefulthink
Copy link
Collaborator

Thanks a lot. That's an error-message I haven't yet seen, and a very helpful one at that.
Let me see if I can figure out how it's even possible that lat/lng are Infinity. I'll also try to reproduce it myself.

@ronlv10
Copy link
Author

ronlv10 commented Nov 10, 2024

great, thanks a lot!

If you're struggling to reproduce let me know. I would be happy to jump on a call, I got to reproduce it through a few zooming.

@ronlv10
Copy link
Author

ronlv10 commented Nov 17, 2024

Hey @usefulthink , Have you able to reproduce it on your side ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants