Use machine-readable lat/lon format

diff --git a/app/assets/javascripts/index/note.js.erb b/app/assets/javascripts/index/note.js.erb
index 8c2ab5c5671f8d629e9a083eb3212f0e16c335e0..1f9dbe37538313ccbf0ef30b229cd6ecdad4cc9a 100644 (file)
--- a/app/assets/javascripts/index/note.js.erb
+++ b/app/assets/javascripts/index/note.js.erb
@@ -61,16 +61,17 @@ OSM.Note = function (map) {
 
     content.find("textarea").val('').trigger("input");
 
-    var data = $('.details').data();
+    var data = $('.details').data(),
+      latLng = data.coordinates.split(',');
+
     if (!window.location.hash) {
-      var coords = data.coordinates.split(',');
       OSM.route.moveListenerOff();
       map.once('moveend', OSM.route.moveListenerOn);
-      map.getZoom() > 15 ? map.panTo(coords) : map.setView(coords, 16);
+      map.getZoom() > 15 ? map.panTo(latLng) : map.setView(latLng, 16);
     }
 
     if (!map.hasLayer(halo)) {
-      halo = L.circleMarker(data.coordinates.split(','), {
+      halo = L.circleMarker(latLng, {
         weight: 2.5,
         radius: 20,
         fillOpacity: 0.5,
@@ -80,7 +81,7 @@ OSM.Note = function (map) {
     }
 
     if (map.hasLayer(currentNote)) map.removeLayer(currentNote);
-    currentNote = L.marker(data.coordinates.split(','), {
+    currentNote = L.marker(latLng, {
       icon: noteIcons[data.status],
       opacity: 1,
       clickable: true

diff --git a/app/views/browse/note.html.erb b/app/views/browse/note.html.erb
index 90d34b5742dbbd8ea6a3c5f2e735efc132327faa..967487af64136fb74a893442f6ac8429cca3b329 100644 (file)
--- a/app/views/browse/note.html.erb
+++ b/app/views/browse/note.html.erb
@@ -11,7 +11,7 @@
     <%= h(@note.comments.first.body.to_html) %>
   </div>
 
-  <div class="details" data-coordinates="<%= number_with_delimiter(@note.lat) %>,<%=number_with_delimiter(@note.lon) %>" data-status="<%= @note.status %>">
+  <div class="details" data-coordinates="<%= @note.lat %>,<%= @note.lon %>" data-status="<%= @note.status %>">
     <%= note_event('open', @note.created_at, @note.author) %>
     <% if @note.status == "closed" %>
       <br/>