Skip to content

Conversation

@igoroctaviano
Copy link
Collaborator

@igoroctaviano igoroctaviano commented Nov 24, 2025

  • When zoom is unlimited, compare the current resolution to the resolution at the original max zoom level (or the highest available if maxZoom exceeds the number of resolutions). This aligns with the limited-zoom behavior.

The fix:
Uses the resolution value at the maxZoom index (or the last resolution if maxZoom is too high)
Compares currentResolution <= targetResolution (smaller resolution = higher zoom in OpenLayers)
This should prevent detecting high resolution too early. The console will show the target resolution being used for comparison, which should help verify the behavior.

@igoroctaviano
Copy link
Collaborator Author

@fedorov, please test this before merging!

@fedorov
Copy link
Member

fedorov commented Nov 26, 2025

@igoroctaviano as discussed in Slack, I would appreciate it if you could create a separate PR disabling overzooming so I can test both approaches.

@igoroctaviano igoroctaviano changed the title Add unlimite zoom handling ANN clustering: Add unlimited zoom handling Nov 26, 2025
@igoroctaviano igoroctaviano changed the title ANN clustering: Add unlimited zoom handling [Draft] ANN clustering: Add unlimited zoom handling Nov 26, 2025
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.

3 participants