Skip to content

docs(whats-new): Add basemap enhancements to whats-new.md#10218

Merged
chrisgervang merged 6 commits intomasterfrom
copilot/extract-basemap-enhancements
Apr 14, 2026
Merged

docs(whats-new): Add basemap enhancements to whats-new.md#10218
chrisgervang merged 6 commits intomasterfrom
copilot/extract-basemap-enhancements

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 14, 2026

Ports the basemap/integration documentation improvements to whats-new.md.

Background

The chr/mapbox-widget-support branch contained both widget support work and basemap/integration improvements to whats-new.md. This PR adds only the basemap-related doc changes, leaving all existing sections (order, names, and content) unchanged. No PR links are included in the new entries.

Change List

  • Expand @deck.gl/mapbox section — add four new entries (no PR links):
    • In interleaved mode, MapboxOverlay now always renders layers in groups by beforeId or slot, enabling cross-layer extension handling by default.
    • Multi-view setups now work consistently across overlaid and interleaved modes.
    • Basemap now works correctly when canvas has zero dimensions.
    • Heatmap layer now blends correctly in interleaved mode.
  • Add ### @deck.gl/google-maps section — overlaid layers now render at the correct DOM position when interleaved: false.
Original prompt

Could you please extract the what's new basemap engancements parts that are missing on master from chr/mapbox-widget-support which are already implemented? (So, everything except the widget support).

Please match the existing writing style and organizations


Note

Low Risk
Low risk: documentation-only changes that don’t affect runtime behavior.

Overview
Updates docs/whats-new.md (deck.gl v9.3) to expand the @deck.gl/mapbox release notes with additional basemap/integration fixes (multi-view consistency, zero-dimension canvas handling, and heatmap blending in interleaved mode), and adds a new @deck.gl/google-maps section noting corrected DOM positioning for overlaid layers when interleaved: false.

Reviewed by Cursor Bugbot for commit 6991972. Bugbot is set up for automated code reviews on this repo. Configure here.

…support

Agent-Logs-Url: https://github.com/visgl/deck.gl/sessions/8abc470f-79a4-48e7-877b-b8632ffc9ce8

Co-authored-by: chrisgervang <2461547+chrisgervang@users.noreply.github.com>
Copilot AI changed the title [WIP] Extract what's new basemap enhancements from chr/mapbox-widget-support docs(whats-new): Extract basemap enhancements from chr/mapbox-widget-support Apr 14, 2026
Copilot AI requested a review from chrisgervang April 14, 2026 01:51
Comment thread docs/whats-new.md
- All widget documentation pages now have live demos and easy-to-follow code samples.
- A bug was fixed where widgets used with the `DeckGL` React component did not block pointer interaction with the canvas underneath.

### Views and Controllers
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Don't change this section. Stick to basemap changes only

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Restored the ### Layers section to its original state (Tile3DLayer bullet is back). Removed the duplicate from ### Improved 3D Support. (93ae003)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Don't reorder anything either

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Restored the original section order (Views and ControllersLayersExtensions@deck.gl/mapbox). (a1244de)

…ile3DLayer from Improved 3D Support

Agent-Logs-Url: https://github.com/visgl/deck.gl/sessions/90bf2889-cce8-4259-91ab-4ea59739f823

Co-authored-by: chrisgervang <2461547+chrisgervang@users.noreply.github.com>
Comment thread docs/whats-new.md
- All widget documentation pages now have live demos and easy-to-follow code samples.
- A bug was fixed where widgets used with the `DeckGL` React component did not block pointer interaction with the canvas underneath.

### Views and Controllers
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Don't reorder anything either

Comment thread docs/whats-new.md Outdated
Comment on lines +75 to +58
### Extensions
### @deck.gl/extensions
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Don't rename sections

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Restored ### Extensions and ### Views and Controllers to their original names. (a1244de)

Comment thread docs/whats-new.md Outdated
In interleaved mode, `MapboxOverlay` now always renders layers in groups by `beforeId` or `slot`. This enables cross-layer extension handling (e.g. MaskExtension, CollisionFilterExtension) by default, without needing the previously experimental `_renderLayersInGroups` prop.
- In interleaved mode, `MapboxOverlay` now always renders layers in groups by `beforeId` or `slot`. This enables cross-layer extension handling (e.g. MaskExtension, CollisionFilterExtension) by default, without needing the previously experimental `_renderLayersInGroups` prop. ([#10163](https://github.com/visgl/deck.gl/pull/10163))
- `MapboxOverlay` now automatically injects a `MapView` with id `"mapbox"` when custom views are provided, so overlaid and interleaved modes behave the same way with multi-view setups. ([#9947](https://github.com/visgl/deck.gl/pull/9947))
- Fixed null viewport handling when the map canvas has zero dimensions. ([#10076](https://github.com/visgl/deck.gl/pull/10076), [#10086](https://github.com/visgl/deck.gl/pull/10086))
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We should a phrasing like "X now works correctly" for bug fixes. Like "basemap now works correctly when canvas has zero dimensions"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated to "Basemap now works correctly when canvas has zero dimensions." (a1244de)

Comment thread docs/whats-new.md Outdated
- In interleaved mode, `MapboxOverlay` now always renders layers in groups by `beforeId` or `slot`. This enables cross-layer extension handling (e.g. MaskExtension, CollisionFilterExtension) by default, without needing the previously experimental `_renderLayersInGroups` prop. ([#10163](https://github.com/visgl/deck.gl/pull/10163))
- `MapboxOverlay` now automatically injects a `MapView` with id `"mapbox"` when custom views are provided, so overlaid and interleaved modes behave the same way with multi-view setups. ([#9947](https://github.com/visgl/deck.gl/pull/9947))
- Fixed null viewport handling when the map canvas has zero dimensions. ([#10076](https://github.com/visgl/deck.gl/pull/10076), [#10086](https://github.com/visgl/deck.gl/pull/10086))
- Fixed heatmap layer blending in interleaved mode. ([#9993](https://github.com/visgl/deck.gl/pull/9993))
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

"Heatmap layer now blends correctly in interleaved mode"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated to "Heatmap layer now blends correctly in interleaved mode." (a1244de)

Comment thread docs/whats-new.md Outdated

### @deck.gl/google-maps

- Fixed DOM positioning when `interleaved: false`. ([#9992](https://github.com/visgl/deck.gl/pull/9992))
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Say the scenario that was fixed, not the fix

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated to "Overlaid layers now render at the correct DOM position when interleaved: false." (a1244de)

Copilot AI changed the title docs(whats-new): Extract basemap enhancements from chr/mapbox-widget-support docs(whats-new): Add basemap enhancements to whats-new.md Apr 14, 2026
Copilot AI requested a review from chrisgervang April 14, 2026 17:19
Comment thread docs/whats-new.md Outdated

In interleaved mode, `MapboxOverlay` now always renders layers in groups by `beforeId` or `slot`. This enables cross-layer extension handling (e.g. MaskExtension, CollisionFilterExtension) by default, without needing the previously experimental `_renderLayersInGroups` prop.
- In interleaved mode, `MapboxOverlay` now always renders layers in groups by `beforeId` or `slot`. This enables cross-layer extension handling (e.g. MaskExtension, CollisionFilterExtension) by default, without needing the previously experimental `_renderLayersInGroups` prop. ([#10163](https://github.com/visgl/deck.gl/pull/10163))
- `MapboxOverlay` now automatically injects a `MapView` with id `"mapbox"` when custom views are provided, so overlaid and interleaved modes behave the same way with multi-view setups. ([#9947](https://github.com/visgl/deck.gl/pull/9947))
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this should be shortened to just talk about what's changed. the first part is getting into the what but similar to the other ones we prefer to just focus on the use case change.

Also remove PR links from these additions

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Shortened to "Multi-view setups now work consistently across overlaid and interleaved modes." and removed PR links from all three new additions. (ddb669e)

…ions

Agent-Logs-Url: https://github.com/visgl/deck.gl/sessions/6c69aefe-b3a4-4a76-9362-3777bd02d829

Co-authored-by: chrisgervang <2461547+chrisgervang@users.noreply.github.com>
Comment thread docs/whats-new.md Outdated
### @deck.gl/mapbox

In interleaved mode, `MapboxOverlay` now always renders layers in groups by `beforeId` or `slot`. This enables cross-layer extension handling (e.g. MaskExtension, CollisionFilterExtension) by default, without needing the previously experimental `_renderLayersInGroups` prop.
- In interleaved mode, `MapboxOverlay` now always renders layers in groups by `beforeId` or `slot`. This enables cross-layer extension handling (e.g. MaskExtension, CollisionFilterExtension) by default, without needing the previously experimental `_renderLayersInGroups` prop. ([#10163](https://github.com/visgl/deck.gl/pull/10163))
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please no PR links

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed the PR link from that entry. (6991972)

Agent-Logs-Url: https://github.com/visgl/deck.gl/sessions/4e3bfd0f-e3d6-4595-a54a-167dec9667b5

Co-authored-by: chrisgervang <2461547+chrisgervang@users.noreply.github.com>
@chrisgervang chrisgervang marked this pull request as ready for review April 14, 2026 23:18
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6991972. Configure here.

Comment thread docs/whats-new.md
### @deck.gl/mapbox

In interleaved mode, `MapboxOverlay` now always renders layers in groups by `beforeId` or `slot`. This enables cross-layer extension handling (e.g. MaskExtension, CollisionFilterExtension) by default, without needing the previously experimental `_renderLayersInGroups` prop.
- In interleaved mode, `MapboxOverlay` now always renders layers in groups by `beforeId` or `slot`. This enables cross-layer extension handling (e.g. MaskExtension, CollisionFilterExtension) by default, without needing the previously experimental `_renderLayersInGroups` prop.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Reviewer feedback to shorten entry not addressed

Low Severity

The reviewer explicitly requested this entry be shortened to "just talk about what's changed," noting "the first part is getting into the what but similar to the other ones we prefer to just focus on the use case change." The PR description itself contains the shorter intended text ("enabling cross-layer extension handling by default"), but the committed line still includes implementation details about specific extensions (MaskExtension, CollisionFilterExtension) and the _renderLayersInGroups prop. Only the missing - bullet prefix was fixed; the content was not shortened as requested.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 6991972. Configure here.

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 81.588%. remained the same — copilot/extract-basemap-enhancements into master

@chrisgervang chrisgervang merged commit 7ec4f4d into master Apr 14, 2026
6 checks passed
@chrisgervang chrisgervang deleted the copilot/extract-basemap-enhancements branch April 14, 2026 23:44
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