Skip to content
This repository has been archived by the owner on Dec 9, 2021. It is now read-only.

Some lat,longtidue do not merge to bigger cluster. #252

Closed
tvhieu242 opened this issue Jul 22, 2021 · 2 comments · Fixed by #265
Closed

Some lat,longtidue do not merge to bigger cluster. #252

tvhieu242 opened this issue Jul 22, 2021 · 2 comments · Fixed by #265
Assignees
Labels

Comments

@tvhieu242
Copy link

tvhieu242 commented Jul 22, 2021

As u can see the image below: the 148 marker do not merge to biggest cluster ...
(other markers have longer distance had merged but this 148)
image
expected:
image

So i had checked it and find out the problem from lat,longitude
there are 4 markers make this issue (if i change these lat,longitude then the issue gone)
-38.670073 178.026234
-38.667111 178.024509
-38.670353 178.023302
-38.659139 178.016418
and they located in Gisborne
image

is there any problem with Gisborne?

@hanmuk-oh
Copy link
Contributor

hanmuk-oh commented Sep 2, 2021

Hi @tvhieu242, I found the same issue when clustering the map pin in Gisborne!.

Maybe you could try this walkaround by removing the if statement and else clause in the below code.

if (this.getMap().getZoom() > 3) { 
  mapBounds = new google.maps.LatLngBounds(
    (this.getMap() as google.maps.Map).getBounds().getSouthWest(),
    (this.getMap() as google.maps.Map).getBounds().getNorthEast()
  );
} else { 
  mapBounds = new google.maps.LatLngBounds(
    new google.maps.LatLng(85.02070771743472, -178.48388434375),
    new google.maps.LatLng(-85.08136444384544, 178.00048865625)
  );
}

if (this.getMap().getZoom() > 3) {

I guess there was a bug on Google Map API, but looks like we don't need to limit the bound when the zoom level is less than 3 anymore.

@github-actions
Copy link

🎉 This issue has been resolved in version 1.2.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants