Merge remote-tracking branch 'openstreetmap/pull/1445'

[rails.git] / app / assets / javascripts / index / contextmenu.js

diff --git a/app/assets/javascripts/index/contextmenu.js b/app/assets/javascripts/index/contextmenu.js
index 8a4654b97b9316b336e1ff70e8f031e9273aed71..b19c5d34f4f728a1b7b534c094cd577bf5221ece 100644 (file)
--- a/app/assets/javascripts/index/contextmenu.js
+++ b/app/assets/javascripts/index/contextmenu.js
@@ -30,9 +30,12 @@ OSM.initializeContextMenu = function (map) {
   map.contextmenu.addItem({
     text: I18n.t("javascripts.context.add_note"),
     callback: function addNoteHere(e) {
   map.contextmenu.addItem({
     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);

     }
   });
 
     }
   });
 


@@ -68,7 +71,7 @@ OSM.initializeContextMenu = function (map) {
   });
 
   map.on("mousedown", function (e) {
   });
 
   map.on("mousedown", function (e) {

-    if (e.shiftKey) map.contextmenu.disable();
+    if (e.originalEvent.shiftKey) map.contextmenu.disable();

   }).on("mouseup", function () {
     map.contextmenu.enable();
   });
   }).on("mouseup", function () {
     map.contextmenu.enable();
   });