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

Set maxzoom to 20 by default #1531

Closed
peterqliu opened this issue May 12, 2015 · 6 comments
Closed

Set maxzoom to 20 by default #1531

peterqliu opened this issue May 12, 2015 · 6 comments
Labels
GL JS parity For feature parity with Mapbox GL JS

Comments

@peterqliu
Copy link
Contributor

for parity with gl-js

snippet

@1ec5
Copy link
Contributor

1ec5 commented May 12, 2015

Does gl-js have a library-level maximum zoom level, or does it only care what the tile source says the maximum is?

@1ec5 1ec5 added the GL JS parity For feature parity with Mapbox GL JS label May 12, 2015
@peterqliu
Copy link
Contributor Author

@1ec5 1ec5 changed the title Set maxzoom to 20 Set maxzoom to 20 by default May 12, 2015
@1ec5
Copy link
Contributor

1ec5 commented May 12, 2015

That seems like a reasonable way to do it. Anyone know why it isn’t configurable in the first place?

@kkaefer
Copy link
Member

kkaefer commented May 13, 2015

It was never implemented. No reason not to implement it though.

@mb12
Copy link

mb12 commented May 13, 2015

Currently Collison::max placement depth is hard coded to 3 i.e. for a source with Max zoom 15, Symbols are placed until zoom 18. This limitation will go away once @ansis changes for label placement are merged into master.

@jfirebaugh
Copy link
Contributor

Duplicate of #509.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
GL JS parity For feature parity with Mapbox GL JS
Projects
None yet
Development

No branches or pull requests

5 participants