Skip to content

Add an option to MarkerClusterGroup to pass options to L.Polygon#68

Merged
danzel merged 1 commit intoLeaflet:masterfrom
yohanboniface:polygonOptions
Sep 25, 2012
Merged

Add an option to MarkerClusterGroup to pass options to L.Polygon#68
danzel merged 1 commit intoLeaflet:masterfrom
yohanboniface:polygonOptions

Conversation

@yohanboniface
Copy link
Member

It could be useful for example to customize the color and fillColor.

Another solution, by the way more powerful, could be to provide the possibility to define the constructor method (say clusterToPolygon), like it's done in Leaflet with geometryToLayer. But this last was more intrusive.

What do you think?

Thanks!

…Polygon

It could be useful for example to customize the color and fillColor.
@danzel
Copy link
Member

danzel commented Sep 24, 2012

Awesome, will pull this later today

danzel added a commit that referenced this pull request Sep 25, 2012
Add an option to MarkerClusterGroup to pass options to L.Polygon
@danzel danzel merged commit 41237f6 into Leaflet:master Sep 25, 2012
@yohanboniface
Copy link
Member Author

Thanks! :)
Do you want me to mention this option in the README or it's too late?

@yohanboniface
Copy link
Member Author

Oh, you've already done it, nice :)

@danzel
Copy link
Member

danzel commented Sep 25, 2012

Yep :)
Was going to ask you to do it but I remembered the readme was a total mess, so I fixed that up while I was in there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants