Merge branch 'pull/4985'

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

diff --git a/app/assets/javascripts/index/new_note.js b/app/assets/javascripts/index/new_note.js
index 55c5cbc934314f31670289d629833cb2313959d6..887ba043b12b00bda5478b4e2dfd3aeec305efd3 100644 (file)
--- a/app/assets/javascripts/index/new_note.js
+++ b/app/assets/javascripts/index/new_note.js
@@ -1,10 +1,12 @@
-OSM.NewNote = function(map) {
+//= require qs/dist/qs
+
+OSM.NewNote = function (map) {

   var noteLayer = map.noteLayer,
   var noteLayer = map.noteLayer,

-    content = $('#sidebar_content'),
-    page = {},
-    addNoteButton = $(".control-note .control-button"),
-    newNote,
-    halo;
+      content = $("#sidebar_content"),
+      page = {},
+      addNoteButton = $(".control-note .control-button"),
+      newNote,
+      halo;

 
   var noteIcons = {
     "new": L.icon({
 
   var noteIcons = {
     "new": L.icon({


@@ -28,9 +30,9 @@ OSM.NewNote = function(map) {
     e.preventDefault();
     e.stopPropagation();
 
     e.preventDefault();
     e.stopPropagation();
 

-    if ($(this).hasClass('disabled')) return;
+    if ($(this).hasClass("disabled")) return;

 
 

-    OSM.router.route('/note/new');
+    OSM.router.route("/note/new");

   });
 
   function createNote(marker, form, url) {
   });
 
   function createNote(marker, form, url) {


@@ -61,7 +63,7 @@ OSM.NewNote = function(map) {
       newNote = null;
       noteLayer.removeLayer(marker);
       addNoteButton.removeClass("active");
       newNote = null;
       noteLayer.removeLayer(marker);
       addNoteButton.removeClass("active");

-      OSM.router.route('/note/' + feature.properties.id);
+      OSM.router.route("/note/" + feature.properties.id);

     }
   }
 
     }
   }
 


@@ -69,7 +71,7 @@ OSM.NewNote = function(map) {
     var marker = L.marker(feature.geometry.coordinates.reverse(), {
       icon: noteIcons[feature.properties.status],
       opacity: 0.9,
     var marker = L.marker(feature.geometry.coordinates.reverse(), {
       icon: noteIcons[feature.properties.status],
       opacity: 0.9,

-      clickable: true
+      interactive: true

     });
     marker.id = feature.properties.id;
     marker.addTo(noteLayer);
     });
     marker.id = feature.properties.id;
     marker.addTo(noteLayer);


@@ -77,14 +79,18 @@ OSM.NewNote = function(map) {
   }
 
   page.pushstate = page.popstate = function (path) {
   }
 
   page.pushstate = page.popstate = function (path) {

-    OSM.loadSidebarContent(path, page.load);
+    OSM.loadSidebarContent(path, function () {
+      page.load(path);
+    });

   };
 
   function newHalo(loc, a) {
   };
 
   function newHalo(loc, a) {

-    if (a === 'dragstart' && map.hasLayer(halo)) {
+    var hasHalo = halo && map.hasLayer(halo);
+
+    if (a === "dragstart" && hasHalo) {

       map.removeLayer(halo);
     } else {
       map.removeLayer(halo);
     } else {

-      if (map.hasLayer(halo)) map.removeLayer(halo);
+      if (hasHalo) map.removeLayer(halo);

 
       halo = L.circleMarker(loc, {
         weight: 2.5,
 
       halo = L.circleMarker(loc, {
         weight: 2.5,


@@ -97,7 +103,7 @@ OSM.NewNote = function(map) {
     }
   }
 
     }
   }
 

-  page.load = function () {
+  page.load = function (path) {

     if (addNoteButton.hasClass("disabled")) return;
     if (addNoteButton.hasClass("active")) return;
 
     if (addNoteButton.hasClass("disabled")) return;
     if (addNoteButton.hasClass("active")) return;
 


@@ -105,24 +111,26 @@ OSM.NewNote = function(map) {
 
     map.addLayer(noteLayer);
 
 
     map.addLayer(noteLayer);
 

-    var mapSize = map.getSize();
-    var markerPosition;
+    var params = Qs.parse(path.substring(path.indexOf("?") + 1));
+    var markerLatlng;

 
 

-    if (mapSize.y > 800) {
-      markerPosition = [mapSize.x / 2, mapSize.y / 2];
-    } else if (mapSize.y > 400) {
-      markerPosition = [mapSize.x / 2, 400];
+    if (params.lat && params.lon) {
+      markerLatlng = L.latLng(params.lat, params.lon);

     } else {
     } else {

-      markerPosition = [mapSize.x / 2, mapSize.y];
+      markerLatlng = map.getCenter();

     }
 
     }
 

-    newNote = L.marker(map.containerPointToLatLng(markerPosition), {
-      icon: noteIcons["new"],
+    map.panInside(markerLatlng, {
+      padding: [50, 50]
+    });
+
+    newNote = L.marker(markerLatlng, {
+      icon: noteIcons.new,

       opacity: 0.9,
       draggable: true
     });
 
       opacity: 0.9,
       draggable: true
     });
 

-    newNote.on("dragstart dragend", function(a) {
+    newNote.on("dragstart dragend", function (a) {

       newHalo(newNote.getLatLng(), a.type);
     });
 
       newHalo(newNote.getLatLng(), a.type);
     });
 


@@ -131,8 +139,6 @@ OSM.NewNote = function(map) {
 
     newNote.on("remove", function () {
       addNoteButton.removeClass("active");
 
     newNote.on("remove", function () {
       addNoteButton.removeClass("active");

-    }).on("dragstart",function () {
-      $(newNote).stopTime("removenote");

     }).on("dragend", function () {
       content.find("textarea").focus();
     });
     }).on("dragend", function () {
       content.find("textarea").focus();
     });


@@ -145,17 +151,17 @@ OSM.NewNote = function(map) {
       $(e.target.form.add).prop("disabled", $(e.target).val() === "");
     }
 
       $(e.target.form.add).prop("disabled", $(e.target).val() === "");
     }
 

-    content.find('input[type=submit]').on('click', function (e) {
+    content.find("input[type=submit]").on("click", function (e) {

       e.preventDefault();
       e.preventDefault();

-      createNote(newNote, e.target.form, '/api/0.6/notes.json');
+      createNote(newNote, e.target.form, "/api/0.6/notes.json");

     });
 
     return map.getState();
   };
 
   page.unload = function () {
     });
 
     return map.getState();
   };
 
   page.unload = function () {

-    noteLayer.removeLayer(newNote);
-    map.removeLayer(halo);
+    if (newNote) noteLayer.removeLayer(newNote);
+    if (halo) map.removeLayer(halo);

     addNoteButton.removeClass("active");
   };
 
     addNoteButton.removeClass("active");
   };