From 1ccf5b308afeeb3b48526221acd5f6d761ba301f Mon Sep 17 00:00:00 2001
From: Tatiana Smirnova <tatiana.smirnova@nopio.com>
Date: Fri, 30 Aug 2024 00:34:28 +0200
Subject: [PATCH] fix: APPS-2896 Styling: Ursus Item Detail View - keywords not
 aligned (#1153)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Andy Wallace <andrewwallace@library.ucla.edu>
---
 .../_keyword_metadata.html.erb                | 24 +++++++++----------
 1 file changed, 11 insertions(+), 13 deletions(-)

diff --git a/app/views/catalog/work_record--ursus/_keyword_metadata.html.erb b/app/views/catalog/work_record--ursus/_keyword_metadata.html.erb
index 55a9428c6..e9a1c38f9 100644
--- a/app/views/catalog/work_record--ursus/_keyword_metadata.html.erb
+++ b/app/views/catalog/work_record--ursus/_keyword_metadata.html.erb
@@ -18,19 +18,17 @@
           <% end %>
         <% end %>
         <% if (@document[:subject_tesim].present? && !@document[:subject_tesim].empty?) || (document[:subject_topic_tesim].present? && !@document[:subject_topic_tesim].empty?) %>
-          <dt class="blacklight-subjects metadata-block__label-key">
-          <!-- KEY -->
-          Subjects
-          <!-- VALUE -->
-            <% keyword.each do |field_name, field| %>
-              <% if field_name == "subject_tesim" || field_name == "subject_topic_tesim" %>
-                <dd class="blacklight-<%= field_name.parameterize %> metadata-block__label-value metadata-block__label-value--ursus">
-                  <%= doc_presenter.field_value field %>
-                </dd>
-              <% end %>
-            <% end %>
-          </dt>
-
+          <% keyword.select { |field_name, _field| field_name == "subject_tesim" || field_name == "subject_topic_tesim" }
+                    .each_with_index do |(field_name, field), i| %>
+            <!-- KEY -->
+            <dt class="blacklight-subjects metadata-block__label-key">
+              <%= 'Subjects' if i.zero? %>
+            </dt>
+            <!-- VALUE -->
+            <dd class="blacklight-<%= field_name.parameterize %> metadata-block__label-value metadata-block__label-value--ursus">
+              <%= doc_presenter.field_value field %>
+            </dd>
+          <% end %>
         <% end %>
       </dl>
       <hr class='divider divider--ursus'>