-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Underzoom example [ch106065] #4
base: master
Are you sure you want to change the base?
Conversation
Question: should this example be included right now (for the first set of examples published at dev-center?). If that's the case we need a couple of small changes:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A confirmation of when + 2 small changes would be needed
minZoom: 12, | ||
maxZoom: 16, | ||
|
||
// Allow underzooming by setting the bounding box of the layer. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need more clarification, something like this: by setting minZoom, maxZoom and autoExtent, the layer will be visible for zoom>16 but it will always use z=16 tiles, for zoom<12 the layer will be visible but it always use z=12 tiles
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👌
pointRadiusMinPixels: 6, | ||
lineWidthMinPixels: 1, | ||
|
||
// Define the zoom range of the layer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the tiles ranges
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you mean?
Not sure about when it should be included, but at least we may have to wait until the new |
Related to https://app.clubhouse.io/cartoteam/story/106065/overzoom
It requires
autoExtent
implementation from https://github.com/CartoDB/deck.gl/tree/feature/ch106065/autoExtent