fix: supercluster max zoom with fractional zooming

[scarg0e]

Whether to re-cluster on zoom is determined by the fractional zoom value here:

js-markerclusterer/src/algorithms/supercluster.ts

Line 76 in c5cdf54

if (this.state.zoom <= this.maxZoom || state.zoom <= this.maxZoom) {

however the clusters are calculated using a rounded integer value here:

js-markerclusterer/src/algorithms/supercluster.ts

Line 92 in c5cdf54

.getClusters([-180, -90, 180, 90], Math.round(map.getZoom()))

This means if our new zoom value is fractionally above our maxZoom, it will be the final re-cluster on zooming in. However the clusters returned will not be de-clustered, as the rounded down value used for getClusters is not above our maxZoom, leaving our clusters not in the final de-clustered state even as we continue to zoom in.

Fixes #647 🦕

example of existing behaviour from issue 647:

maxZoom=14
zoom map to 14.2
getClusters called for zoom=14
zoom map to 16.2
getClusters not called
Result: will never decluster as getClusters is never called with a value above maxZoom

example of behavior with fix:

maxZoom=14
zoom map to 14.2
getClusters called for zoom=14
zoom map to 16.2
getClusters called for zoom=16
Result: does decluster as getClusters is called with a value above maxZoom

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[lupus78]

I just tracked down this issue, and found this PR. When is this going to be merged into a release?
IMHO, this is pretty big usability issue. It took me days to finally realize what the issue was.