Possible fix?

MITLibraries · Jun 6, 2024 · c1c9f6f · c1c9f6f
1 parent 98caf53
commit c1c9f6f
Show file tree

Hide file tree

Showing 2 changed files with 59 additions and 53 deletions.
diff --git a/app/assets/stylesheets/partials/_results.scss b/app/assets/stylesheets/partials/_results.scss
@@ -41,49 +41,53 @@
 }
 
 .result {
-  padding: 2rem;
   border-top: 1px solid $brand-primary;
   font-size: $fs-small;
-  line-height: 0;
+  line-height: 0; /* line height needs to be reset so iOS doesn't act weird */
 
   &::marker {
     font-size: 2.5rem;
     font-weight: $fw-bold;
   }
 
-  .record-title {
-    font-size: 2.5rem;
-    line-height: 1.1;
-  }
+  .result-content {
+    padding: 2rem;
+    line-height: 1.2;
 
-  &:hover,
-  &:focus {
-    background-color: $gray-l4;
-  }
+    .record-title {
+      font-size: 2.5rem;
+      line-height: 1.2;
+    }
 
-  a:visited {
-    color: $brand-secondary;
-  }
+    &:hover,
+    &:focus {
+      background-color: $gray-l4;
+    }
 
-  .pub-info {
-    font-size: $fs-base;
-    color:  $gray-d1;
-    span:first-child:after {
-      content: " | ";
+    a:visited {
+      color: $brand-secondary;
     }
-    margin: 1em 0;
-  }
 
-  .result-summary {
-    margin-bottom: 2.4em;
-  }
+    .pub-info {
+      font-size: $fs-base;
+      color:  $gray-d1;
+      span:first-child:after {
+        content: " | ";
+      }
+      margin: 1em 0;
+    }
+
+    .result-summary {
+      margin-bottom: 2.4em;
+    }
 
-  .result-highlights {
-    margin-top: 1.4em;
-    ul {
-      list-style: none;
-      li {
-        margin-left: 2rem;
+    .result-highlights {
+      margin-top: 1.4em;
+      ul {
+        list-style: none;
+        li {
+          margin-left: 2rem;
+        }
       }
     }
   }

diff --git a/app/views/search/_result_geo.html.erb b/app/views/search/_result_geo.html.erb
@@ -1,32 +1,34 @@
 <li class="result">
-  <h3 class="record-title">
-    <span class="sr">Title: </span><%= link_to(result_geo['title'], record_path(result_geo['timdexRecordId'])) %>
-  </h3>
+  <div class="result-content">
+    <h3 class="record-title">
+      <span class="sr">Title: </span><%= link_to(result_geo['title'], record_path(result_geo['timdexRecordId'])) %>
+    </h3>
 
-  <div class="data-info">
-    <%= render partial: 'shared/geo_data_info', locals: { metadata: result_geo } %>
-  </div>
+    <div class="data-info">
+      <%= render partial: 'shared/geo_data_info', locals: { metadata: result_geo } %>
+    </div>
 
-  <% if result_geo['contributors'] %> 
-    <span class="sr">Contributors: </span>
-    <ul class="list-inline truncate-list contributors">
-      <%= render partial: 'shared/contributors', locals: { contributors: result_geo['contributors'] } %>
-    </ul>
-  <% end %>
+    <% if result_geo['contributors'] %> 
+      <span class="sr">Contributors: </span>
+      <ul class="list-inline truncate-list contributors">
+        <%= render partial: 'shared/contributors', locals: { contributors: result_geo['contributors'] } %>
+      </ul>
+    <% end %>
 
-  <% if result_geo['summary'] %>
-    <p class="result-summary truncate-list">
-      <span class="sr">Summary: </span><%= result_geo['summary'].join(' ') %>
-    </p>
-  <% end %>
+    <% if result_geo['summary'] %>
+      <p class="result-summary truncate-list">
+        <span class="sr">Summary: </span><%= result_geo['summary'].join(' ') %>
+      </p>
+    <% end %>
 
-  <% if result_geo['highlight'] %>
-    <div class="result-highlights">
-      <%= render partial: 'search/highlights', locals: { result: result_geo } %>
-    </div>
-  <% end %>
+    <% if result_geo['highlight'] %>
+      <div class="result-highlights">
+        <%= render partial: 'search/highlights', locals: { result: result_geo } %>
+      </div>
+    <% end %>
 
-  <div class="result-record">
-    <%= view_record(result_geo['timdexRecordId']) %>
+    <div class="result-record">
+      <%= view_record(result_geo['timdexRecordId']) %>
+    </div>
   </div>
 </li>