cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Add distance/time summary

Browse source
This commit is contained in:
Richard Fairhurst 2014-03-08 16:14:14 +00:00
parent 9a05a7ba49
commit 3ef8024b1d
6 changed files with 23 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

14
app/assets/javascripts/routing.js.erb
View file

@ -194,7 +194,10 @@ OSM.Routing=function(map,name,jqSearch) {
$('#sidebar_content').empty();
var html=('<h2><a class="geolink" href="#" onclick="$(~.close_directions~).click();return false;">' +
'<span class="icon close"></span></a>' + I18n.t('javascripts.directions.directions') +
'</h2><table id="turnbyturn" />').replace(/~/g,"'");
'</h2><p id="routing_summary">' +
I18n.t('javascripts.directions.distance') + ': ' + r.formatDistance(data.distance)+ '. ' +
I18n.t('javascripts.directions.time' ) + ': ' + r.formatTime(data.time) + '.</p>' +
'<table id="turnbyturn" />').replace(/~/g,"'");
$('#sidebar_content').html(html);
// Add each row
var cumulative=0;
@ -224,6 +227,15 @@ OSM.Routing=function(map,name,jqSearch) {
r.clickTurn=function(num,latlng) {
r.popup=L.popup().setLatLng(latlng).setContent("<p>"+(num+1)+"</p>").openOn(r.map);
};
r.formatDistance=function(m) {
if (m < 1000 ) { return Math.round(m) + "m"; }
else if (m < 10000) { return (m/1000.0).toFixed(1) + "km"; }
else { return Math.round(m / 1000) + "km"; }
};
r.formatTime=function(s) {
var d=new Date(s*1000); var h=d.getHours(); var m=d.getMinutes();
return h+":"+(m<10 ? '0' : '')+m;
};
// Close all routing UI

2
app/assets/javascripts/routing_engines/graphhopper.js
View file

@ -44,7 +44,7 @@ GraphHopperEngine.prototype.createConfig = function() {
var distInMeter = instr.distances[i];
steps.push([{lat: latlng[0], lng: latlng[1]}, instrCode, instrText, distInMeter]);
}
router.setItinerary({steps: steps});
router.setItinerary({ steps: steps, distance: data.route.distance, time: data.route['time']/1000 });
return true;
},
GH_INSTR_MAP: {

4
app/assets/javascripts/routing_engines/mapquest.js
View file

@ -47,7 +47,7 @@ MapQuestEngine.prototype.createConfig = function() {
url+="&"+that.vehicleParam;
//url+="&locale=" + I18n.currentLocale(); //Doesn't actually work. MapQuest requires full locale e.g. "de_DE", but I18n only provides language, e.g. "de"
url+="&manMaps=false";
url+="&shapeFormat=raw&generalize=0";
url+="&shapeFormat=raw&generalize=0&unit=k";
this.requestCORS(url);
},
gotRoute: function(router,data) {
@ -69,7 +69,7 @@ MapQuestEngine.prototype.createConfig = function() {
var d=(i==mq.length-1) ? 15: this.MQ_SPRITE_MAP[s.turnType];
steps.push([L.latLng(s.startPoint.lat, s.startPoint.lng), d, s.narrative, s.distance*1000]);
}
router.setItinerary( { steps: steps });
router.setItinerary( { steps: steps, distance: data.route.distance*1000, time: data.route['time'] });
return true;
}
};

2
app/assets/javascripts/routing_engines/osrm.js
View file

@ -46,7 +46,7 @@ OSRMEngine.prototype.createConfig = function() {
if (instCodes[0]!=15) { instText+=s[1] ? "<b>"+s[1]+"</b>" : I18n.t('javascripts.directions.instructions.unnamed'); }
steps.push([line[s[3]], s[0].split('-')[0], instText, s[2]]);
}
if (steps.length) router.setItinerary({ steps: steps });
if (steps.length) router.setItinerary({ steps: steps, distance: data.route_summary.total_distance, time: data.route_summary.total_time });
return true;
}
};

4
app/assets/stylesheets/common.css.scss
View file

@ -639,6 +639,10 @@ nav.secondary {
font-size: 13px;
}
p {
padding: 0 $lineheight $lineheight/4;
}
.icon.close {
float: right;
cursor: pointer;

2
config/locales/en.yml
View file

@ -2127,6 +2127,7 @@ en:
mapquest_foot: "Foot (MapQuest)"
osrm_car: "Car (OSRM)"
directions: "Directions"
distance: "Distance"
errors:
no_route: "Couldn't find a route between those two places."
no_place: "Sorry - couldn't find that place."
@ -2149,6 +2150,7 @@ en:
against_oneway: "Go against one-way on "
end_oneway: "End of one-way on "
unnamed: "(unnamed)"
time: "Time"
redaction:
edit:
description: "Description"