carbonplan / maps

interactive multi-dimensional data-driven web maps
https://maps.demo.carbonplan.org
Other
214 stars 18 forks source link

call setZoom before setCenter #127

Closed iskandari closed 1 week ago

iskandari commented 3 weeks ago

Addresses unexpected behavior:

changing the latitude value in setCenter() has no effect on the map view if setZoom() is called after it, so the order of operations should be inverted

vercel[bot] commented 3 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **maps-demo** | ⬜️ Ignored ([Inspect](https://vercel.com/carbonplan/maps-demo/HhXrFaHSaHGBitTG5AooWnNNv2nh)) | [Visit Preview](https://maps-demo-git-fork-iskandari-bugfix-mapprops-carbonplan.vercel.app) | | Nov 5, 2024 9:34pm | | **maps-docs** | ⬜️ Ignored ([Inspect](https://vercel.com/carbonplan/maps-docs/DYAyj7hQwbkWEBKfjvjTvwrpzPPN)) | [Visit Preview](https://maps-docs-git-fork-iskandari-bugfix-mapprops-carbonplan.vercel.app) | | Nov 5, 2024 9:34pm |
Shane98c commented 1 week ago

released in 3.4.1