From 35e815449d87ffd27d2d9dc0101e22aecc15f296 Mon Sep 17 00:00:00 2001
From: Martin Artnik <martin@scinote.net>
Date: Mon, 30 Aug 2021 13:00:52 +0200
Subject: [PATCH] Experiment dropdown layout fixes [SCI-6013]

---
 .../stylesheets/shared_styles/elements/dropdown.scss |  4 ++--
 .../show/_experiment_actions_dropdown.html.erb       | 12 +++++++++---
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/app/assets/stylesheets/shared_styles/elements/dropdown.scss b/app/assets/stylesheets/shared_styles/elements/dropdown.scss
index 4be60fc47..c11b89aab 100644
--- a/app/assets/stylesheets/shared_styles/elements/dropdown.scss
+++ b/app/assets/stylesheets/shared_styles/elements/dropdown.scss
@@ -41,8 +41,7 @@
 
 .dropdown-menu {
   .form-dropdown-break hr {
-    margin-bottom: 8px;
-    margin-top: 0;
+    margin: 0;
   }
 
   .form-dropdown-item {
@@ -62,6 +61,7 @@
 
     .form-dropdown-item-info {
       color: $color-silver-chalice !important;
+      padding-top: .3em;
     }
 
     .project-archive-restore-form {
diff --git a/app/views/projects/show/_experiment_actions_dropdown.html.erb b/app/views/projects/show/_experiment_actions_dropdown.html.erb
index 1fcb1d3b0..cc1570720 100644
--- a/app/views/projects/show/_experiment_actions_dropdown.html.erb
+++ b/app/views/projects/show/_experiment_actions_dropdown.html.erb
@@ -63,9 +63,15 @@
           <% end %>
         </li>
       <% end %>
-      <li class="form-dropdown-break">
-        <hr>
-      </li>
+      <% if experiment.active? && can_manage_experiment?(experiment) ||
+            can_move_experiment?(experiment) ||
+            can_clone_experiment?(experiment) ||
+            experiment.active? && can_archive_experiment?(experiment) ||
+            experiment.archived? && can_restore_experiment?(experiment) %>
+        <li class="form-dropdown-break">
+          <hr>
+        </li>
+      <% end %>
       <li class="form-dropdown-item">
         <div class="form-dropdown-item-info">
           <small><%= t('experiments.experiment_id') %>: <strong><%= experiment.code %></strong></small>