Changeset boundary is enlarged if it was on a single point

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

diff --git a/app/assets/javascripts/index/history.js b/app/assets/javascripts/index/history.js
index 27774f05b55476eddf0cc5fe628c061e0a944e45..71a94aef0d1ca014d446fa59c8a4ebeaa74c4038 100644 (file)
--- a/app/assets/javascripts/index/history.js
+++ b/app/assets/javascripts/index/history.js
@@ -1,3 +1,5 @@
+//= require jquery.simulate
+
 OSM.History = function(map) {
   var page = {};
 
@@ -9,10 +11,15 @@ OSM.History = function(map) {
     .on("mouseout", "[data-changeset]", function () {
       unHighlightChangeset($(this).data("changeset").id);
     })
-    .on("click", "[data-changeset]", function (e) {
-      if (!$(e.target).is('a')) {
-        clickChangeset($(this).data("changeset").id, e);
-      }
+    .on("mousedown", "[data-changeset]", function () {
+      var moved = false;
+      $(this).one("click", function (e) {
+        if (!moved && !$(e.target).is('a')) {
+          clickChangeset($(this).data("changeset").id, e);
+        }
+      }).one("mousemove", function () {
+        moved = true;
+      });
     });
 
   var group = L.featureGroup()
@@ -41,33 +48,10 @@ OSM.History = function(map) {
   }
 
   function clickChangeset(id, e) {
-    var evt, el = $("#changeset_" + id).find("a.changeset_id")[0];
-    if ('createEvent' in document) {
-      evt = document.createEvent('MouseEvents');
-      evt.initMouseEvent('click',
-        true, // canBubble
-        true, // cancelable
-        window, // 'AbstractView'
-        e.clicks, // click count
-        e.screenX, // screenX
-        e.screenY, // screenY
-        e.clientX, // clientX
-        e.clientY, // clientY
-        e.ctrlKey, // ctrl
-        e.altKey, // alt
-        e.shiftKey, // shift
-        e.metaKey, // meta
-        e.button, // mouse button
-        e.relatedTarget // relatedTarget
-      );
-      el.dispatchEvent(evt);
-    } else {
-      evt = document.createEventObject();
-      el.fireEvent('onclick', evt);
-    }
+    $("#changeset_" + id).find("a.changeset_id").simulate("click", e);
   }
 
-  function loadData() {
+  function update() {
     var data = {list: '1'};
 
     if (window.location.pathname === '/history') {
@@ -78,11 +62,16 @@ OSM.History = function(map) {
       url: window.location.pathname,
       method: "GET",
       data: data,
-      success: function(html, status, xhr) {
+      success: function(html) {
         $('#sidebar_content .changesets').html(html);
         updateMap();
       }
     });
+
+    var feedLink = $('link[type="application/atom+xml"]'),
+      feedHref = feedLink.attr('href').split('?')[0];
+
+    feedLink.attr('href', feedHref + '?bbox=' + data.bbox);
   }
 
   function loadMore(e) {
@@ -108,9 +97,17 @@ OSM.History = function(map) {
     $("[data-changeset]").each(function () {
       var changeset = $(this).data('changeset');
       if (changeset.bbox) {
+        var latModifier = 0;
+        var lonModifier = 0;
+
+        if (changeset.bbox.minlat === changeset.bbox.maxlat || changeset.bbox.minlon === changeset.bbox.maxlon) {
+          latModifier = 0.0001;
+          lonModifier = 0.0002;
+        }
+
         changeset.bounds = L.latLngBounds(
-          [changeset.bbox.minlat, changeset.bbox.minlon],
-          [changeset.bbox.maxlat, changeset.bbox.maxlon]);
+          [changeset.bbox.minlat - latModifier, changeset.bbox.minlon - lonModifier],
+          [changeset.bbox.maxlat + latModifier, changeset.bbox.maxlon + lonModifier]);
         changesets.push(changeset);
       }
     });
@@ -142,18 +139,15 @@ OSM.History = function(map) {
     map.addLayer(group);
 
     if (window.location.pathname === '/history') {
-      map.on("moveend", loadData)
+      map.on("moveend", update);
     }
 
-    loadData();
+    update();
   };
 
   page.unload = function() {
     map.removeLayer(group);
-
-    if (window.location.pathname === '/history') {
-      map.off("moveend", loadData)
-    }
+    map.off("moveend", update);
 
     $("#history_tab").removeClass("current");
   };