From: Andy Allan <git@gravitystorm.co.uk>
Date: Wed, 24 Feb 2021 10:49:39 +0000 (+0000)
Subject: Merge branch 'pull/3091'
X-Git-Tag: live~2740
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/3debff60ef50158c5439003ad4357d1f3dcc120b?hp=a0dd71112a910f68383392bc9efaa26b8e042d3e

Merge branch 'pull/3091'
---

diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index 481025d81..a8f138caa 100644
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -975,6 +975,16 @@ tr.turn:hover {
   }
 }
 
+/* Bootstrap buttons don't have any vertical margin, so
+   they touch when adjacent buttons wrap onto a new line
+   e.g. wide form buttons on a narrow sidebar */
+
+.btn-wrapper {
+  > .btn {
+    margin-bottom: $spacer * 0.25;
+  }
+}
+
 /* Rules for export sidebar */
 
 .export_form {
diff --git a/app/views/browse/note.html.erb b/app/views/browse/note.html.erb
index 1a792e873..6dcd46e76 100644
--- a/app/views/browse/note.html.erb
+++ b/app/views/browse/note.html.erb
@@ -51,7 +51,7 @@
         <div class="form-group">
           <textarea class="form-control" name="text" cols="40" rows="5" maxlength="2000"></textarea>
         </div>
-        <div>
+        <div class="btn-wrapper">
           <% if current_user.moderator? -%>
             <input type="submit" name="hide" value="<%= t("javascripts.notes.show.hide") %>" class="btn btn-light" data-note-id="<%= @note.id %>" data-method="DELETE" data-url="<%= note_url(@note, "json") %>">
           <% end -%>
@@ -63,7 +63,7 @@
   <% else %>
     <form action="#">
       <input type="hidden" name="text" value="">
-      <div>
+      <div class="btn-wrapper">
         <% if current_user and current_user.moderator? -%>
           <input type="submit" name="hide" value="<%= t("javascripts.notes.show.hide") %>" class="btn btn-light" data-note-id="<%= @note.id %>" data-method="DELETE" data-url="<%= note_url(@note, "json") %>">
         <% end -%>