Skip to content
This repository has been archived by the owner on Feb 28, 2023. It is now read-only.

out-of-box support for Mapbox Studio layers #114

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

severak
Copy link

@severak severak commented May 11, 2016

In current uMap it's impossible to use custom Mapbox Studio layer due technical differences.

Mapbox Studio layers exposes only 512x512 tiles with -1 zoomOffset.

See https://www.mapbox.com/api-documentation/#retrieve-raster-tiles-from-styles

@yohanboniface
Copy link
Member

Thanks!
Not sure that's the best fix though. Maybe we can add those parameter in the tilelayer form instead?
With default values, that should not hurt, given that this form is advanced, so people dealing with it should not be frightened by those more options.

@severak
Copy link
Author

severak commented May 11, 2016

My solution is IMHO more novice-friendly. Mapbox Studio has no mention about these technical differences in "Share" (and use) screen.

I copied tile URL to uMap and wondered why it's so shrinked.

mapboxstudio

@severak
Copy link
Author

severak commented Jun 20, 2016

Hi - Mapbox just added possibility to return 256×256 tiles via their Styles API, see docs. This patch is thus no longer needed (for Mapbox).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants