Skip to content
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

Write fill and outline in KML PolyStyle #10646

Merged
merged 1 commit into from
Mar 31, 2020
Merged

Conversation

mike-000
Copy link
Contributor

Fixes #10614
Fixes #10631

Write styles based on style objects appropriate for geometry.
Write fill and outline in PolyStyle as required (i.e. if set to false)
Various fixes to handle MultiGeometry correctly.

Write styles based on style objects appropriate for geometry.
Write fill and outline in PolyStyle if false (i.e. non-default)
Handle MultiLineString, MultiPoint and MultiPolygon within heterogenous MultiGeometry when writing features

Add getGeometriesArrayRecursive method to ol/geom/GeometryCollection to allow for nested MultiGeometry

Enhanced write GeometryCollection geometries test

A more rigorous write GeometryCollection geometries test including nested collections (the output is simplified to a single MultiGeomtry)

Add writeFeatures to outline and fill tests, setting geometry for geometry specific tests

Add <fill>0</fill> and <outline>0</outline> to some existing tests
Copy link
Member

@ahocevar ahocevar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mike-000!

@ahocevar ahocevar merged commit 2401e0a into openlayers:master Mar 31, 2020
@mike-000 mike-000 deleted the patch-1 branch April 23, 2020 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants