[openstreetmap/openstreetmap-website] Fix destinations and roundabouts locale (#1805)
Jamie Guthrie
notifications at github.com
Sun Apr 1 14:48:23 UTC 2018
Done a few changes here.
I refactored the with_destination locale so its just a string that one can append onto the end of an instruction. Hopefully it will still work with different grammar structures in different languages!
I also fixed some ambiguous roundabout instructions..
i.e. https://www.openstreetmap.org/directions?engine=osrm_car&route=53.7338%2C-1.5258%3B53.6832%2C-1.4999#map=13/53.7086/-1.5171
Instructions 7-16 are a bit weird - so I've fixed it to specify `"Exit straight/slight right/right from roundabout"` when exiting roundabouts
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/1805
-- Commit Summary --
* Refactored destinations locale into separate instruction
* Refactored locale templates and fixed roundabout exiting templates
-- File Changes --
M app/assets/javascripts/index/directions/osrm.js (29)
M config/locales/en.yml (80)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/1805.patch
https://github.com/openstreetmap/openstreetmap-website/pull/1805.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/1805
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20180401/8e8c76da/attachment.html>
More information about the rails-dev
mailing list