[openstreetmap-website] show total route distance in results interface (#710)

danstowell notifications at github.com
Sun Feb 23 21:15:16 UTC 2014


Hi - this pull request is the product of my first poke around with your jsrouting branch. It simply puts the cumulative distance into the results GUI.

Distance is shown in km - I'm not aware of any osm i18n call that would decide whether to output miles/km but maybe that could come later.
You can merge this Pull Request by running:

  git pull https://github.com/danstowell/openstreetmap-website jsrouting

Or you can view, comment on it, or merge it online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/710

-- Commit Summary --

  * Beginning of JavaScript-only routing UI
  * Presentation work
  * Precompile routing JS
  * Presentation of directions
  * Query options presentation
  * Close directions
  * Start to make turn-by-turn instructions clickable
  * Show little popup on click
  * Start work on second engine
  * Move routing engines into their own files
  * Simplify usage, fix closures
  * Start MapQuest Open support
  * Move more engine-specific stuff out of main .js
  * Parse MapQuest directions
  * Parse CloudMade directions
  * Allow user to click Go before geocode result
  * Remove logging
  * Start work on drag-and-drop start/end markers
  * More work on draggable markers
  * More draggable routes work
  * Add GraphHopper engine (thanks @karussell!)
  * Integrate marker & geocoding behaviours
  * Apparently 'final' is a reserved word in JS...
  * Repair Go button
  * Remove a couple of TODOs
  * Remove some repeated-query funkiness
  * Make more of the javascript routing functions internationalizable
  * Merge pull request #6 from apmon/jsroute
  * Fiiiix siiiiilly direeeeections
  * output total distance

-- File Changes --

    A app/assets/images/routing-sprite.png (0)
    M app/assets/javascripts/index.js (46)
    A app/assets/javascripts/routing.js.erb (269)
    A app/assets/javascripts/routing_engines/cloudmade_foot.js (42)
    A app/assets/javascripts/routing_engines/graphhopper_bicycle.js (48)
    A app/assets/javascripts/routing_engines/mapquest_bicycle.js (65)
    A app/assets/javascripts/routing_engines/osrm_car.js (42)
    M app/assets/stylesheets/common.css.scss (46)
    M app/views/layouts/_search.html.erb (22)
    M config/environments/production.rb (2)
    M config/locales/de.yml (32)
    M config/locales/en.yml (32)
    A vendor/assets/leaflet/leaflet.polyline.js (127)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/710.patch
https://github.com/openstreetmap/openstreetmap-website/pull/710.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/710
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20140223/759ce928/attachment.html>


More information about the rails-dev mailing list