simplegeo/polymaps — Issue #111: Enable fill:none for LineString objects drawn by po.geoJson
Author: natevw
Created: 2012-01-24 06:15:21+00:00
State: open

Right now the geoJson layer draws both LineString and Polygon objects using the SVG \ element. When dealing with heterogeneous or unknown/generic feature sets (such as one might find in a GeometryCollection) this makes it difficult to style filled polygons without also ending up with "filled" lines, since SVG always applies an implicit "close path" when a fill is specified.

I'd like the mapping from GeoJSON to SVG to work as follows:

* point → circle
* linestring → polyline
* polygon → path
* multi → g
* collection → g

Then I could e.g. style all polyline elements in the map with fill:none and not have to deal with special classing/whatnot in a JS stylist workaround.

For my purposes (https://github.com/natevw/Metakaolin/issues/1) I could write a simple enough — non-tiled, non-data-fetching — custom layer to draw geometries in the above fashion, but I'm wondering if this is a reasonable change to submit to trunk on po.geoJson itself?