A couple of additions to Hanbyuls lrm branch

nextzen / lrm-mapzen

Support for Mapzen Turn-by-Turn routing in Leaflet Routing Machine

https://mapzen.com/projects/turn-by-turn/

Other

86 stars 31 forks source link

A couple of additions to Hanbyuls lrm branch #44

Closed kdiluca closed 8 years ago

kdiluca commented 8 years ago

The refactor looks great! Additions include:

updated route polyline base color
added walking style updates & line break markers (I think this a nice to have but it's up to you on if you want to keep it)
fixed "travel" typo
renaming of transit color function to be more generic

FYI: I'm going to use this to replace the transit product demo one since this is much cleaner and I'll submit a PR to that team. Thanks!!

kdiluca commented 8 years ago

@hanbyul-here After reviewing this refactor, we can simplify the logic even more. Let me try and I'll add to this PR. Also, this should fix that issue that I added as well for transfers.

hanbyul-here commented 8 years ago

code became much cleaner, thanks :)