X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/430978fab7c842d87992ff7d984b16943e66a6fe..91c30d3a8c0b746cafa91e09740981ed682e4fa7:/app/assets/javascripts/index/contextmenu.js

diff --git a/app/assets/javascripts/index/contextmenu.js b/app/assets/javascripts/index/contextmenu.js
index f9f49db15..1ee703e60 100644
--- a/app/assets/javascripts/index/contextmenu.js
+++ b/app/assets/javascripts/index/contextmenu.js
@@ -1,6 +1,6 @@
 OSM.initializeContextMenu = function (map) {
   map.contextmenu.addItem({
-    text: "Directions from here",
+    text: I18n.t("javascripts.context.directions_from"),
     callback: function directionsFromHere(e) {
       var precision = OSM.zoomPrecision(map.getZoom()),
           latlng = e.latlng.wrap(),
@@ -8,13 +8,14 @@ OSM.initializeContextMenu = function (map) {
           lng = latlng.lng.toFixed(precision);
 
       OSM.router.route("/directions?" + querystring.stringify({
-        route: lat + "," + lng + ";" + $("#route_to").val()
+        from: lat + "," + lng,
+        to: $("#route_to").val()
       }));
     }
   });
 
   map.contextmenu.addItem({
-    text: "Directions to here",
+    text: I18n.t("javascripts.context.directions_to"),
     callback: function directionsToHere(e) {
       var precision = OSM.zoomPrecision(map.getZoom()),
           latlng = e.latlng.wrap(),
@@ -22,22 +23,26 @@ OSM.initializeContextMenu = function (map) {
           lng = latlng.lng.toFixed(precision);
 
       OSM.router.route("/directions?" + querystring.stringify({
-        route: $("#route_from").val() + ";" + lat + "," + lng
+        from: $("#route_from").val(),
+        to: lat + "," + lng
       }));
     }
   });
 
   map.contextmenu.addItem({
-    text: "Add a note here",
+    text: I18n.t("javascripts.context.add_note"),
     callback: function addNoteHere(e) {
-      // I'd like this, instead of panning, to pass a query parameter about where to place the marker
-      map.panTo(e.latlng.wrap(), {animate: false});
-      OSM.router.route("/note/new");
+      var precision = OSM.zoomPrecision(map.getZoom()),
+          latlng = e.latlng.wrap(),
+          lat = latlng.lat.toFixed(precision),
+          lng = latlng.lng.toFixed(precision);
+
+      OSM.router.route("/note/new?lat=" + lat + "&lon=" + lng);
     }
   });
 
   map.contextmenu.addItem({
-    text: "Show address",
+    text: I18n.t("javascripts.context.show_address"),
     callback: function describeLocation(e) {
       var precision = OSM.zoomPrecision(map.getZoom()),
           latlng = e.latlng.wrap(),
@@ -49,7 +54,7 @@ OSM.initializeContextMenu = function (map) {
   });
 
   map.contextmenu.addItem({
-    text: "Query features",
+    text: I18n.t("javascripts.context.query_features"),
     callback: function queryFeatures(e) {
       var precision = OSM.zoomPrecision(map.getZoom()),
           latlng = e.latlng.wrap(),
@@ -61,16 +66,15 @@ OSM.initializeContextMenu = function (map) {
   });
 
   map.contextmenu.addItem({
-    text: "Centre map here",
+    text: I18n.t("javascripts.context.centre_map"),
     callback: function centreMap(e) {
       map.panTo(e.latlng);
     }
   });
 
   map.on("mousedown", function (e) {
-    if (e.shiftKey) map.contextmenu.disable();
-  }).on("mouseup", function () {
-    map.contextmenu.enable();
+    if (e.originalEvent.shiftKey) map.contextmenu.disable();
+    else map.contextmenu.enable();
   });
 
   var updateMenu = function updateMenu () {