update default tile/style endpoints for mapbox styles

riastrad commented 4 years ago

Resolves #1316

Up until now, Mapbox.js defaults have relied heavily on the Mapbox Legacy Static Tiles endpoints.

I'll mark this PR as ready for review when the following are implemented:

[x] Updates initialization code so that passing in a template mapID will automatically load the equivalent style from the modern Static Tiles API.
[x] Set console warning when automatic support for classic style IDs (e.g. riastrad.{class style id}) is passed to map objects and tileLayer objects.
[x] ~Logs a console warning about dropping support for the outdated tileLayer.setFormat() method with default mapIDs.~ Given that the url pattern for legacy & modern APIs is different, this appears to be unnecessary.
[x] All previous tests are updated and passing
[x] new tests for changes to default behavior

If I identify any other usage of legacy APIs within the defaults, I will flag it and incorporate it within this PR before the review.

cc @mapbox/static-apis

riastrad commented 4 years ago

👋 @mapbox/gl-js @mapbox/static-apis this branch changes some pretty core functionality, so would appreciate a close review from one or both of you.

riastrad commented 4 years ago

👋 @asheemmamoowala @mourner any chance you'll be able to review this before the end of the week?

mourner commented 4 years ago

@riastrad yes, will do!

mapbox / mapbox.js