From d0fc8202087a2f21b857f7c1d87ea9abb9d64161 Mon Sep 17 00:00:00 2001
From: Shana Moore <shana@scientist.com>
Date: Wed, 6 Dec 2023 14:47:23 -0800
Subject: [PATCH] Update show.html.erb

---
 .../cultural_show/hyrax/base/show.html.erb    | 70 +++++++------------
 1 file changed, 26 insertions(+), 44 deletions(-)

diff --git a/app/views/themes/cultural_show/hyrax/base/show.html.erb b/app/views/themes/cultural_show/hyrax/base/show.html.erb
index 9c5117b69..83c869d5d 100644
--- a/app/views/themes/cultural_show/hyrax/base/show.html.erb
+++ b/app/views/themes/cultural_show/hyrax/base/show.html.erb
@@ -11,66 +11,48 @@
       <div class="panel-body">
         <div class="row">
           <%= render 'workflow_actions_widget', presenter: @presenter %>
-          <% if @presenter.video_embed_viewer? %>
-            <%= render 'video_embed_viewer', presenter: @presenter %>
-          <% elsif @presenter.iiif_viewer? %>
-            <div class="col-sm-12 centered-media">
-              <%= render 'representative_media', presenter: @presenter, viewer: true %>
-            </div>
-          <% elsif @presenter.show_pdf_viewer? %>
-            <div class="col-sm-12">
-              <%= render 'pdf_js', file_set_presenter: pdf_file_set_presenter(@presenter) %>
-            </div>
-          <% end %>
           <div class="col-sm-12">
             <%= render "show_actions", presenter: @presenter %>
           </div>
-          <div class="centered-media <%= !@presenter.iiif_viewer? ? 'col-sm-6 text-center' : '' %>">
-            <%= render 'representative_media', presenter: @presenter, viewer: false unless @presenter.iiif_viewer? || @presenter.show_pdf_viewer? %>
-          <% if !@presenter.iiif_viewer? %>
-            <div>
-            <br/>
-              <%= render('download_pdf', presenter: @presenter, file_set_id: @presenter.file_set_presenters.first.id) if @presenter.show_pdf_download_button? %>
-              <%= render 'citations', presenter: @presenter %>
-              <!-- analytics_button is disabled until future fix -->
-            <%#= render 'analytics_button', presenter: @presenter %>
-            <br/>
-            </div>
+          <% if @presenter.video_embed_viewer? %>
+            <%= render 'video_embed_viewer', presenter: @presenter %>
+          <% else %>
+            <% if @presenter.iiif_viewer? %>
+              <div class="col-sm-12 centered-media">
+                <%= render 'representative_media', presenter: @presenter, viewer: true %>
+              </div>
+            <% elsif @presenter.pdf_viewer? %>
+              <div class="col-sm-12">
+                <%= render 'pdf_js', file_set_presenter: pdf_file_set_presenter(@presenter) %>
+              </div>
+            <% else %>
+              <div class="centered-media <%= !@presenter.viewer? ? 'col-sm-6 text-center' : '' %>">
+                <%= render 'representative_media', presenter: @presenter, viewer: false %>
+              </div>
+            <% end %>
           <% end %>
-          </div>
-          <div class="<%= @presenter.iiif_viewer? ? 'col-sm-8' : 'col-sm-6' %>">
+          <div class="<%= @presenter.viewer? ? 'col-sm-8' : 'col-sm-8' %>">
             <%= render 'work_description', presenter: @presenter %>
             <%= render 'metadata', presenter: @presenter %>
           </div>
-          <div class="<%= @presenter.iiif_viewer? ? 'col-sm-4' : 'col-sm-12' %> relationships-container">
+          <div class="col-sm-12 col-md-4 relationships-container">
             <div class='relationships-box'>
               <%= render 'relationships', presenter: @presenter %>
-              <% if @presenter.class == Hyrax::OerPresenter %>
-                <%= render 'related_items', presenter: @presenter %>
-              <% end %>
             </div>
-            <% if @presenter.iiif_viewer? %>
-              <div class="center-block">
-              <br>
-                <%= render('download_pdf', presenter: @presenter, file_set_id: @presenter.file_set_presenters.first.id) if @presenter.show_pdf_download_button? %>
-                <%= render 'citations', presenter: @presenter %>
-                <!-- analytics_button is disabled until future fix -->
+            <%= render('download_pdf', presenter: @presenter, file_set_id: @presenter.file_set_presenters.first.id) if @presenter.show_pdf_download_button? %>
+            <%= render 'citations', presenter: @presenter %>
+            <!-- analytics_button is disabled until future fix -->
             <%#= render 'analytics_button', presenter: @presenter %>
-              </div>
-            <% end %>
           </div>
           <div class="col-sm-12 work-show-items">
             <%= render 'items', presenter: @presenter %>
-            <%# TODO: we may consider adding these partials in the future %>
-            <%# = render 'sharing_with', presenter: @presenter %>
-            <%# = render 'user_activity', presenter: @presenter %>
           </div>
+          <span class='hide analytics-event' data-category="work" data-action="work-view" data-name="<%= @presenter.id %>" >
+          <% @presenter.member_of_collection_ids.each do |collection_id| %>
+            <span class='hide analytics-event' data-category="work-in-collection" data-action="work-in-collection-view" data-name="<%= collection_id %>" >
+          <% end %>
         </div>
       </div>
     </div>
-    <span class='hide analytics-event' data-category="work" data-action="work-view" data-name="<%= @presenter.id %>" >
-    <% @presenter.member_of_collection_ids.each do |collection_id| %>
-      <span class='hide analytics-event' data-category="work-in-collection" data-action="work-in-collection-view" data-name="<%= collection_id %>" >
-    <% end %>
   </div>
-</div>
+</div>
\ No newline at end of file