[openstreetmap/openstreetmap-website] Re add valhalla (PR #3848)

[Richard Fairhurst]

Had a play around with this and it looks great.

One suggestion: the Graphhopper and OSRM instructions write out an index (in bold) at the start of each step. So:

> **3.** Turn right onto **Enstone Road**	90m

rather than just

> Turn right onto **Enstone Road**	90m

For consistency and for usability it would be good to include this in the Valhalla instructions. (It's particularly useful when discussing a shared route, e.g. "why is Valhalla sending me along Frog Street at step 7 on this route".)

Otherwise looks good to go.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/3848#issuecomment-1362787863
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/3848/c1362787863 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20221222/2b42778b/attachment.htm>