Skip to content

Fix Cesium point zoom with map padding - #13052

Open
offtherailz wants to merge 1 commit into
masterfrom
codex/fix-cesium-point-padding
Open

offtherailz wants to merge 1 commit into
masterfrom
codex/fix-cesium-point-padding

Conversation

@offtherailz

Copy link
Copy Markdown
Member

Fixes #13050.

Change

  • Resolve the configured max zoom before applying Cesium map padding to a point extent.
  • Keep the existing point radius behavior when no finite max zoom is available.
  • Cover both cases with Cesium zoom hook regression tests.

Verification

  • The two new tests failed before the fix with DeveloperError: normalized result is not a number.
  • Focused CesiumMap Karma suite: 26 passed, 1 pre-existing skip.
  • npm run lint passed.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Attribute Table - Zoom to extent on 3D mode

2 participants