Use MapboxOverlay
for handling Maplibre interaction?
#437
+52
−17
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change list
import "maplibre-gl/dist/maplibre-gl.css";
. This is required to show the attribution control. Fixes Import MapLibre CSS #436.MapboxOverlay
for synchronizing Deck and Maplibre.With these changes, it looks much more like a mapbox/maplibre map the user might be familiar with (note the ScaleControl, NavigationControl, FullscreenControl and AttributionControl):
There's a wider question here: should Maplibre be a child of Deck or should Deck be a child of Maplibre? The Deck docs list some limitations:
Maplibre as a child of Deck (not using MapboxOverlay)
MapboxOverlay
you can't have multiple interactive deck views.Deck as a child of Maplibre (using MapboxOverlay)
fly_to
is now broken. This is because deck is no longer maintaining the view state itself.