feat(mapbox): Add widget support to MapboxOverlay via IControl adapter

[chrisgervang]

Closes #9962

Background

Enable deck widgets to coexist with native Mapbox/MapLibre controls without DOM overlap by rendering widgets with viewId: 'mapbox' into the map's control container system via the new _container prop.

Change List

• Add DeckWidgetControl class that wraps deck widgets as Mapbox IControls
• Process widgets with viewId: 'mapbox' in MapboxOverlay, automatically wrapping them as IControls
• Set widget.props._container to Mapbox-positioned element so WidgetManager appends there
• Widgets stack correctly with native controls in the same DOM container, preventing overlap

---

Note

Medium Risk
Moderate risk because it changes MapboxOverlay lifecycle/setProps behavior to dynamically add/remove map controls and mutate widget _container targets, which could affect widget rendering or control cleanup across mapbox/maplibre versions.

Overview
Adds support for rendering deck.gl widgets inside the Mapbox/MapLibre control container when a widget sets viewId: 'mapbox', so widgets can be positioned alongside native controls without DOM overlap.

MapboxOverlay now detects viewId: 'mapbox' widgets, wraps them in a new DeckWidgetControl (an IControl adapter that sets widget.props._container), manages control reuse/removal across setProps, and cleans everything up on onRemove (works in both overlaid and interleaved modes). Styling is adjusted to remove widget margins when embedded in map controls, docs/whats-new are updated, and examples/tests are expanded (including a new widget-browser test app and updated mapbox mocks).

^{Reviewed by Cursor Bugbot for commit 3ef91cd. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coveralls]

coverage: 81.638% (+0.05%) from 81.588% — chr/mapbox-widget-support into master

[cursor]

Hardcoded string instead of imported constant MAPBOX_VIEW_ID

Low Severity

_processWidgets uses the hardcoded string 'mapbox' to filter widgets, but the constant MAPBOX_VIEW_ID is already imported in this same file (line 13) and used for the same conceptual purpose at line 348 in _getViews. Using the literal instead of the constant creates an inconsistency — if MAPBOX_VIEW_ID were ever updated, this filter would silently stop matching.

 

^{Reviewed by Cursor Bugbot for commit b1babd7. Configure here.}

[cursor]

Widget control references diverge from WidgetManager's resolved widgets

Low Severity

When reusing a control for a new widget instance with the same id, setWidget(widget) updates the control to reference the new widget instance. However, WidgetManager._setWidgets independently reuses the old widget instance (calling oldWidget.setProps(widget.props) and keeping oldWidget). This means DeckWidgetControl._widget and the widget actually managed by WidgetManager are different objects. Consequently, DeckWidgetControl.onRemove clears _container on the wrong (unmanaged) widget instance, leaving the managed widget's _container stale and pointing to a removed DOM element.

Additional Locations (1)

• modules/mapbox/src/deck-widget-control.ts#L42-L49

 

^{Reviewed by Cursor Bugbot for commit b1babd7. Configure here.}

[cursor]

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

There are 3 total unresolved issues (including 2 from previous reviews).

^{❌ 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 afec60a. Configure here.}