stamen / maperture

https://stamen.github.io/maperture/
MIT License
39 stars 12 forks source link

update renderer based on sources #184

Closed aparlato closed 1 year ago

aparlato commented 1 year ago

Description

Previously we only checked the style URL to determine if the renderers dropdown needed to be limited to Mapbox GL. This PR checks source URLs as well in that utility function since Mapbox URLs in a sources object also require Mapbox's renderer.

Noting that because this uses the stylesheet after its fetched, there is a brief period where the app will try to render the Mapbox style in Maplibre. There's no real user consequence here, but there will be console errors. Let me know if it's worthwhile to avoid this, but otherwise this seems ok to me.

Closes #183

QA steps

Author checklist

Create the PR

After approval