From 62af5cc723101e06ae20663fa1241430e48edf1e Mon Sep 17 00:00:00 2001
From: Mojca Lorber <mlorber@biosistemika.com>
Date: Tue, 22 Dec 2020 15:50:27 +0100
Subject: [PATCH] [SCI-5287] Show step collapse/expand on task

---
 app/views/protocols/_steps.html.erb | 40 ++++++++++++++---------------
 1 file changed, 19 insertions(+), 21 deletions(-)

diff --git a/app/views/protocols/_steps.html.erb b/app/views/protocols/_steps.html.erb
index eed7b5af5..5337ab948 100644
--- a/app/views/protocols/_steps.html.erb
+++ b/app/views/protocols/_steps.html.erb
@@ -1,28 +1,26 @@
+<div class="row">
+  <div class="expand-all-steps" style="display: <%= 'none' if @protocol.steps.empty? %>">
+    <a class="btn btn-light" data-action="collapse-steps">
+      <span class="fas fa-caret-up"></span>
+      <span><%= t("protocols.steps.collapse_label") %></a></span>
+    <a class="btn btn-light" data-action="expand-steps">
+      <span class="fas fa-caret-down"></span>
+      <span><%= t("protocols.steps.expand_label") %></a></span>
+  </div>
+</div>
 <% if can_manage_protocol_in_module?(@protocol) || can_manage_protocol_in_repository?(@protocol) %>
   <div class="row">
-    <div class="expand-all-steps" style="display: <%= 'none' if @protocol.steps.empty? %>">
-      <a class="btn btn-light" data-action="collapse-steps">
-        <span class="fas fa-caret-up"></span>
-        <span><%= t("protocols.steps.collapse_label") %></a></span>
-      <a class="btn btn-light" data-action="expand-steps">
-        <span class="fas fa-caret-down"></span>
-        <span><%= t("protocols.steps.expand_label") %></a></span>
+    <div class="pull-left" style="margin: 0 0 20px 15px;">
+      <a href="#"
+         class="btn btn-primary"
+         data-action="new-step"
+         data-href="<%= new_protocol_step_path(protocol_id: @protocol.id, format: :json) %>"
+         style="<%= @protocol.steps.empty? ? 'display: none;' : '' %>">
+        <span class="fas fa-plus" aria-hidden="true"></span>
+        <span class="hidden-xs"><%=t "protocols.steps.new_step" %></span>
+      </a>
     </div>
   </div>
-  <% unless @my_module %>
-    <div class="row">
-      <div class="pull-left" style="margin: 0 0 20px 15px;">
-        <a href="#"
-           class="btn btn-primary"
-           data-action="new-step"
-           data-href="<%= new_protocol_step_path(protocol_id: @protocol.id, format: :json) %>"
-           style="<%= @protocol.steps.empty? ? 'display: none;' : '' %>">
-          <span class="fas fa-plus" aria-hidden="true"></span>
-          <span class="hidden-xs"><%=t "protocols.steps.new_step" %></span>
-        </a>
-      </div>
-    </div>
-  <% end %>
 <% end %>
 <div id="steps">
   <% @protocol.steps.order(:position).each do |step| %>