Added new feature to draw routes with multiple points #5

Closed
ArystanK wants to merge 1 commits from routes into main
ArystanK commented 2022-07-17 15:47:28 +03:00 (Migrated from github.com)
No description provided.
altavir (Migrated from github.com) reviewed 2022-07-17 20:51:03 +03:00
@ -40,6 +40,18 @@ abstract class MapCustomFeature(
abstract fun drawFeature(drawScope: DrawScope, offset: Offset)
}
class MapRouteFeature(
altavir (Migrated from github.com) commented 2022-07-17 20:50:56 +03:00

Couldn't it be replaced by combination of lines? Or it has some unique capabilities?

Couldn't it be replaced by combination of lines? Or it has some unique capabilities?
altavir commented 2022-07-19 11:02:42 +03:00 (Migrated from github.com)

I've made some breaking changes to the core features API, could you please resolve conflicts?

I've made some breaking changes to the core features API, could you please resolve conflicts?
ArystanK commented 2022-07-19 15:25:45 +03:00 (Migrated from github.com)

Unnecessary functionality

Unnecessary functionality
altavir commented 2022-07-23 18:11:01 +03:00 (Migrated from github.com)

@ArystanK I think that we actually want this feature, especially if you add additional parameters like PointMode and caps to the feature definition. We can call it route instead of points though. Could you resolve conflicts with the current branch?

@ArystanK I think that we actually want this feature, especially if you add additional parameters like PointMode and caps to the feature definition. We can call it `route` instead of `points` though. Could you resolve conflicts with the current branch?

Pull request closed

Sign in to join this conversation.
No description provided.