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

Use distinct from, to and via markers #129

Closed
bagage opened this issue Aug 17, 2018 · 3 comments · Fixed by #253
Closed

Use distinct from, to and via markers #129

bagage opened this issue Aug 17, 2018 · 3 comments · Fixed by #253
Assignees
Milestone

Comments

@bagage
Copy link
Collaborator

bagage commented Aug 17, 2018

It seems there is no way today to know in which the route is going:

image

It would be nice to have different icon and/or color for start marker and end marker, for instance on osm.org:

image

@nrenner
Copy link
Owner

nrenner commented Aug 17, 2018

I agree, it's just not implemented yet (Turistforeningen/leaflet-routing#8).

An idea was to first evaluate whether it would make sense to switch to leaflet-routing-machine, which is lacking other features.

@bagage
Copy link
Collaborator Author

bagage commented Aug 17, 2018

Thanks for the quick insight @nrenner. The main advantage of leaflet-routing-machine I see is that it is still under development, but I do not know much more about each project and the lacking features.

If it requires a huge work for replacement maybe I'll give leaflet routing icons configuration a try.

@bagage bagage changed the title Display start and stop marker with different icons Use distinct from, to and via markers Aug 6, 2019
@bagage bagage self-assigned this Oct 6, 2019
@nrenner
Copy link
Owner

nrenner commented Oct 17, 2019

I updated the dev branch of my leaflet-routing fork that we reference with your PR. You should be able to update to that with yarn upgrade leaflet-routing.

@bagage bagage closed this as completed Oct 19, 2019
@nrenner nrenner added this to the 0.11.0 milestone Jan 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants