Stylist title bug in case of Geometry collections

simplegeo / polymaps

Polymaps is a free JavaScript library for making dynamic, interactive maps in modern web browsers.

http://polymaps.org/

Other

1.6k stars 213 forks source link

Stylist title bug in case of Geometry collections #105

Open elpaso opened 12 years ago

elpaso commented 12 years ago

There was a bug in Stylist title function when dealing with GEOMETRYCOLLECTION: the title function removed all children, resulting in all child geometries being removed in case of a collection.

This patch makes a check before removing children to create titles and also adds titles to children in case the geometry is a collection.

shawnbot commented 12 years ago

This looks good to me. @mbostock?

(Thanks, Alessandro!)