Merge remote-tracking branch 'upstream/pull/4868'

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

diff --git a/app/assets/javascripts/index/notes.js b/app/assets/javascripts/index/notes.js
index 524b0b55f81adfc5d4fdc6eaa7dcfbe1098ca339..5d553e970c4be49aa9a05f0c6984c1e0361429d3 100644 (file)
--- a/app/assets/javascripts/index/notes.js
+++ b/app/assets/javascripts/index/notes.js
@@ -1,4 +1,4 @@
-function initializeNotes(map) {
+OSM.initializeNotes = function (map) {
   var noteLayer = map.noteLayer,
       notes = {};
 
@@ -21,32 +21,34 @@ function initializeNotes(map) {
   };
 
   map.on("layeradd", function (e) {
-    if (e.layer == noteLayer) {
+    if (e.layer === noteLayer) {
       loadNotes();
       map.on("moveend", loadNotes);
     }
   }).on("layerremove", function (e) {
-    if (e.layer == noteLayer) {
+    if (e.layer === noteLayer) {
       map.off("moveend", loadNotes);
       noteLayer.clearLayers();
       notes = {};
     }
   });
 
-  noteLayer.on('click', function(e) {
+  noteLayer.on("click", function (e) {
     if (e.layer.id) {
-      OSM.router.route('/note/' + e.layer.id);
+      OSM.router.route("/note/" + e.layer.id);
     }
   });
 
-  function updateMarker(marker, feature) {
+  function updateMarker(old_marker, feature) {
+    var marker = old_marker;
     if (marker) {
       marker.setIcon(noteIcons[feature.properties.status]);
     } else {
       marker = L.marker(feature.geometry.coordinates.reverse(), {
         icon: noteIcons[feature.properties.status],
+        title: feature.properties.comments[0].text,
         opacity: 0.8,
-        clickable: true
+        interactive: true
       });
       marker.id = feature.properties.id;
       marker.addTo(noteLayer);
@@ -54,7 +56,7 @@ function initializeNotes(map) {
     return marker;
   }
 
-  noteLayer.getLayerId = function(marker) {
+  noteLayer.getLayerId = function (marker) {
     return marker.id;
   };
 
@@ -93,4 +95,4 @@ function initializeNotes(map) {
       noteLoader = null;
     }
   }
-}
+};