diff --git a/app/views/protocols/index/_protocol_preview_modal_body.html.erb b/app/views/protocols/index/_protocol_preview_modal_body.html.erb
index ea4530bf9..1a927de67 100644
--- a/app/views/protocols/index/_protocol_preview_modal_body.html.erb
+++ b/app/views/protocols/index/_protocol_preview_modal_body.html.erb
@@ -112,9 +112,17 @@
                     <ul class="list-unstyled">
                       <% assets.each do |asset| %>
                         <li>
-                          <%= image_tag preview_asset_path(asset) if asset.is_image? %>
-                          <p><%= truncate(asset.file_file_name,
-                                          length: Constants::FILENAME_TRUNCATION_LENGTH) %></p>
+                          <% if can_view_or_download_step_assets(@protocol) %>
+                            <%= link_to download_asset_path(asset), data: {no_turbolink: true, id: true, status: "asset-present"} do %>
+                              <%= image_tag preview_asset_path(asset) if asset.is_image? %>
+                              <p><%= truncate(asset.file_file_name,
+                                              length: Constants::FILENAME_TRUNCATION_LENGTH) %></p>
+                            <% end %>
+                          <% else %>
+                            <%= image_tag preview_asset_path(asset) if asset.is_image? %>
+                            <p><%= truncate(asset.file_file_name,
+                                            length: Constants::FILENAME_TRUNCATION_LENGTH) %></p>
+                          <% end %>
                         </li>
                       <% end %>
                     </ul>
@@ -122,7 +130,6 @@
                 <% end %>
 
                 <% unless step.checklists.blank? then %>
-                  <hr>
                   <div class="col-xs-12">
                     <% step.checklists.each do |checklist| %>
                       <strong><%= checklist.name %></strong>
diff --git a/app/views/steps/_step.html.erb b/app/views/steps/_step.html.erb
index 7bf224a51..29d36ee8c 100644
--- a/app/views/steps/_step.html.erb
+++ b/app/views/steps/_step.html.erb
@@ -57,6 +57,7 @@
                 <% end %>
             </div>
           <% end %>
+
           <% assets = ordered_assets(step) %>
           <% unless assets.blank? then %>
             <div class="col-xs-12">