Stretch changeset links in history lists

diff --git a/app/assets/javascripts/index/history.js b/app/assets/javascripts/index/history.js
index 7f0fdf8489dfb21627a2ef0fa17330ffaa803009..c20d342c5c3bafcdbfab633cd6160386de428fd2 100644 (file)
--- a/app/assets/javascripts/index/history.js
+++ b/app/assets/javascripts/index/history.js
@@ -10,18 +10,6 @@ OSM.History = function (map) {
     })
     .on("mouseout", "[data-changeset]", function () {
       unHighlightChangeset($(this).data("changeset").id);
-    })
-    .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()

diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index ad0cf48efff64566be9e6a4ec46c961a670bb480..75ed514eeda38a73a7ebb7f57090a9d5b2ace598 100644 (file)
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -742,10 +742,13 @@ tr.turn:hover {
 
 #sidebar .changesets {
   li {
-    cursor: pointer;
-
     &.selected { background: $list-highlight; }
     /* color is derived from changeset bbox fillColor in history.js */
+
+    a:not(.stretched-link), [title] {
+      position: relative;
+      z-index: 1;
+    }
   }
 
   .comments {

diff --git a/app/views/changesets/_changeset.html.erb b/app/views/changesets/_changeset.html.erb
index 40ee4212a246f26dd64534372c535b3e1eec090e..f3aaf38ea52bdbeb2b8b74226a98ca6aabb121e3 100644 (file)
--- a/app/views/changesets/_changeset.html.erb
+++ b/app/views/changesets/_changeset.html.erb
@@ -12,7 +12,7 @@
 
 <%= tag.li :id => "changeset_#{changeset.id}", :data => { :changeset => changeset_data }, :class => "list-group-item" do %>
   <p class="fst-italic">
-    <a class="changeset_id text-dark" href="<%= changeset_path(changeset) %>">
+    <a class="changeset_id text-dark stretched-link" href="<%= changeset_path(changeset) %>">
       <%= changeset.tags["comment"].to_s.presence || t("browse.no_comment") %>
     </a>
   </p>