-
Notifications
You must be signed in to change notification settings - Fork 38
Conversation
Should |
Let's not create |
👍 sounds good. Shall we keep |
Layers can also have class-specific paint properties, which are applied only when the map has a certain class | ||
name set. For example, a layer with a <code>"paint.night"</code> property would have those properties applied | ||
when the map has the <code>"night"</code> class set. | ||
Layers can also have class-specific paint properties, defined with a property like <code>"paint.night"</code> and applied only when the map has a certain class |
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.
Let's just remove this.
@@ -60,6 +107,29 @@ fs.writeFileSync('./dest.min.json', format(style, 0)); | |||
**Returns** `string`, stringified formatted JSON | |||
|
|||
|
|||
### `exports(layers)` |
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.
Can you fix the naming like you did with derefLayers
? Should be groupByLayout
.
function eachLayer(style, callback) { | ||
for (var k in style.layers) { | ||
callback(style.layers[k]); | ||
eachLayer(style.layers[k], callback); |
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 shouldn't be recursive.
@jfirebaugh ^ changes made. Back to you. |
v8.json
and narrative docs.v9.json
, and removepaint.*
.migrations/v9
with deref & declass functionality.