Skip to content

fix(heremap): preserve integer zoom when focusing on markers#20

Merged
KhaledSakr merged 2 commits intomasterfrom
feature/pos-2137
Feb 12, 2025
Merged

fix(heremap): preserve integer zoom when focusing on markers#20
KhaledSakr merged 2 commits intomasterfrom
feature/pos-2137

Conversation

@KhaledSakr
Copy link
Member

@KhaledSakr KhaledSakr commented Feb 12, 2025

What the PR Includes

  • preserve integer zoom when focusing on markers.

Because of how raster tiles work, the tiles are different based on the zoom level. For example: When zoom level is 12, a tile image covers a certain area, when zooming in to level 13 (meaning, the same land area occupies bigger screen space), a new image needs to be fetched to cover the smaller area.

If fractional zooming is allowed, then a level 12.4 or 12.5 zoom is possible. In this case, the level 12 tiles are still used, but they are stretched over bigger screen space, leading to blurriness.

This change ensures that zoom levels are always integers by passing opt_integerZoom=true to setLookAtData when zooming on a marker or a group of markers.

@KhaledSakr
Copy link
Member Author

/publish

@david-paulus-saad
Copy link
Collaborator

RUN URL: https://github.com/impargo/react-here-maps/actions/runs/13290812359
PACKAGE VERSION: 3.0.0-pre-30

@KhaledSakr
Copy link
Member Author

/publish

@david-paulus-saad
Copy link
Collaborator

RUN URL: https://github.com/impargo/react-here-maps/actions/runs/13291197236
PACKAGE VERSION: 3.0.0-pre-31

@KhaledSakr
Copy link
Member Author

/publish

@david-paulus-saad
Copy link
Collaborator

RUN URL: https://github.com/impargo/react-here-maps/actions/runs/13291372178
PACKAGE VERSION: 3.0.0-pre-32

@KhaledSakr KhaledSakr merged commit b7c1836 into master Feb 12, 2025
5 checks passed
@KhaledSakr KhaledSakr deleted the feature/pos-2137 branch February 12, 2025 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants