Skip to content

Conversation

@sissbruecker
Copy link
Contributor

Description

Adds support for showing lines with one or more segments on the map.

Closes #2885
Closes #2886

Marking as draft as this should wait for 25.1.

Type of change

  • Feature

@sonarqubecloud
Copy link

@ugur-vaadin
Copy link
Contributor

This does not seem to directly add MultiLineString support. Should we keep #2886 open or suggest people using multiple lines to achieve it?

@sissbruecker
Copy link
Contributor Author

I did re-read both tickets today and came to the conclusion that there was a misunderstanding what the purpose of LineString / MultiLineString is. In the ticket for LineString it only mentions single lines with start and end point, while it actually also allows creating paths with multiple segments, which is described as use-case in the ticket for MultiLineString. As such I figured it's fine to close both tickets.

Maybe there would be a use case for adding multiple line strings at once using MultiLineString but that's a different use-case than what's described in the ticket.

Copy link
Contributor

@ugur-vaadin ugur-vaadin left a comment

Choose a reason for hiding this comment

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

LGTM.

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.

[map] Add MultiLineStringFeature [map] Add LineStringFeature

3 participants