From e22b9c35f55632d4c8f3f88b59acc8a4a776b316 Mon Sep 17 00:00:00 2001
From: Andy Allan <git@gravitystorm.co.uk>
Date: Thu, 30 Jan 2020 13:14:53 +0100
Subject: [PATCH 1/2] Remove list styling from various sidebar lists

---
 app/views/browse/_node.html.erb     | 2 +-
 app/views/browse/_relation.html.erb | 4 ++--
 app/views/browse/_way.html.erb      | 4 ++--
 app/views/browse/changeset.html.erb | 2 +-
 app/views/browse/note.html.erb      | 2 +-
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/app/views/browse/_node.html.erb b/app/views/browse/_node.html.erb
index 52502ad4b4..6acc5e01ed 100644
--- a/app/views/browse/_node.html.erb
+++ b/app/views/browse/_node.html.erb
@@ -12,7 +12,7 @@
 
     <% unless node.ways.empty? and node.containing_relation_members.empty? %>
       <h4><%= t "browse.part_of" %></h4>
-      <ul>
+      <ul class="list-unstyled">
         <% node.ways.uniq.each do |way| %>
           <li><%= link_to printable_name(way), { :action => "way", :id => way.id.to_s }, { :class => link_class("way", way), :title => link_title(way) } %></li>
         <% end %>
diff --git a/app/views/browse/_relation.html.erb b/app/views/browse/_relation.html.erb
index 4525563645..b54581b8bd 100644
--- a/app/views/browse/_relation.html.erb
+++ b/app/views/browse/_relation.html.erb
@@ -12,12 +12,12 @@
 
     <% unless relation.containing_relation_members.empty? %>
       <h4><%= t "browse.part_of" %></h4>
-      <ul><%= render :partial => "containing_relation", :collection => relation.containing_relation_members.uniq %></ul>
+      <ul class="list-unstyled"><%= render :partial => "containing_relation", :collection => relation.containing_relation_members.uniq %></ul>
     <% end %>
 
     <% unless relation.relation_members.empty? %>
       <h4><%= t ".members" %></h4>
-      <ul><%= render :partial => "relation_member", :collection => relation.relation_members %></ul>
+      <ul class="list-unstyled"><%= render :partial => "relation_member", :collection => relation.relation_members %></ul>
     <% end %>
   </div>
 <% end %>
diff --git a/app/views/browse/_way.html.erb b/app/views/browse/_way.html.erb
index 5560344a86..137d529ff2 100644
--- a/app/views/browse/_way.html.erb
+++ b/app/views/browse/_way.html.erb
@@ -12,14 +12,14 @@
 
     <% unless way.containing_relation_members.empty? %>
       <h4><%= t "browse.part_of" %></h4>
-      <ul>
+      <ul class="list-unstyled">
         <%= render :partial => "containing_relation", :collection => way.containing_relation_members.uniq %>
       </ul>
     <% end %>
 
     <% unless way.way_nodes.empty? %>
       <h4><%= t ".nodes" %></h4>
-      <ul>
+      <ul class="list-unstyled">
         <% way.way_nodes.each do |wn| %>
           <li>
             <%= link_to printable_name(wn.node), { :action => "node", :id => wn.node_id.to_s }, { :class => link_class("node", wn.node), :title => link_title(wn.node), :rel => link_follow(wn.node) } %>
diff --git a/app/views/browse/changeset.html.erb b/app/views/browse/changeset.html.erb
index 76112e41ea..9b68a03e79 100644
--- a/app/views/browse/changeset.html.erb
+++ b/app/views/browse/changeset.html.erb
@@ -30,7 +30,7 @@
   <% if @comments.length > 0 %>
     <div class='changeset-comments'>
       <form action="#">
-        <ul>
+        <ul class="list-unstyled">
           <% @comments.each do |comment| %>
             <% if comment.visible %>
               <li id="c<%= comment.id %>">
diff --git a/app/views/browse/note.html.erb b/app/views/browse/note.html.erb
index 2389598bcb..884f95d579 100644
--- a/app/views/browse/note.html.erb
+++ b/app/views/browse/note.html.erb
@@ -29,7 +29,7 @@
 
   <% if @note_comments.length > 1 %>
     <div class='note-comments'>
-      <ul>
+      <ul class="list-unstyled">
         <% @note_comments[1..-1].each do |comment| %>
           <li id="c<%= comment.id %>">
             <small class='deemphasize'><%= note_event(comment.event, comment.created_at, comment.author) %></small>

From 9ce2822b3dfc55a38ae7b03bcdd11f9ff93d95ee Mon Sep 17 00:00:00 2001
From: Andy Allan <git@gravitystorm.co.uk>
Date: Thu, 30 Jan 2020 13:34:29 +0100
Subject: [PATCH 2/2] Fix border spacing for sidebar tags and routing tables

The reset code for border spacing was removed in 5c933868dbff and
led to minor artifacts in these two tables.
---
 app/assets/stylesheets/common.scss | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index ae3b7c954c..d3382eb3f2 100644
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -902,6 +902,7 @@ header .search_forms,
     padding: 5px 20px 10px 15px;
     width: 100%;
     border-collapse: separate;
+    border-spacing: 0;
 }
 
 div.direction {
@@ -1026,6 +1027,7 @@ tr.turn:hover {
     border-radius: 3px;
     table-layout: fixed;
     border-collapse: separate;
+    border-spacing: 0;
 
     th, td {
       border-bottom: 1px solid $grey;