Skip to content

fix(mapbox): onBefore|AfterRender fires without layers#10166

Merged
felixpalmer merged 4 commits intomasterfrom
felix/mapboxoverlay-no-layers
Apr 2, 2026
Merged

fix(mapbox): onBefore|AfterRender fires without layers#10166
felixpalmer merged 4 commits intomasterfrom
felix/mapboxoverlay-no-layers

Conversation

@felixpalmer
Copy link
Copy Markdown
Collaborator

Background

When using interleaved=true MapboxOverlay the onBeforeRender and onAfterRender hooks not fire when we have no layers

Change List

  • Explicitly fire onBeforeRender & onAfterRender

Copy link
Copy Markdown
Collaborator

@chrisgervang chrisgervang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a unit test for this?

@coveralls
Copy link
Copy Markdown

coveralls commented Apr 2, 2026

Coverage Status

coverage: 80.393% (+0.004%) from 80.389%
when pulling d20da27 on felix/mapboxoverlay-no-layers
into d3ac4a1 on master.

@felixpalmer felixpalmer merged commit e096e80 into master Apr 2, 2026
3 checks passed
@felixpalmer felixpalmer deleted the felix/mapboxoverlay-no-layers branch April 2, 2026 10:00
chrisgervang added a commit that referenced this pull request Apr 2, 2026
Port onBeforeRender/onAfterRender test from #10166 to overlay spec
(master added it to the deleted mapbox-layer.spec.ts).
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