Use the MUI icon for fullscreen on pdf viewer

Fixes #2352
sul-dlss · Feb 6, 2025 · eefac05 · eefac05
1 parent 135560a
commit eefac05
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/app/components/pdf_component.html.erb b/app/components/pdf_component.html.erb
@@ -5,9 +5,12 @@
     <% unless component.requested_by_chromium? %>
       <button
         aria-label="Full screen"
-        class="btn sul-i-expand-1"
+        class="btn"
         data-action="fullscreen#toggle"
         id="full-screen-button">
+          <span class="button-label">
+            <svg class="MuiSvgIcon-root" focusable="false" aria-hidden="true" viewBox="0 0 24 24" tabindex="-1" title="Fullscreen"><path d="M7 14H5v5h5v-2H7zm-2-4h2V7h3V5H5zm12 7h-3v2h5v-5h-2zM14 5v2h3v3h2V5z"></path></svg>
+          </span>
       </button>
     <% end %>
   <% end %>

diff --git a/app/views/layouts/preview/media.html.erb b/app/views/layouts/preview/media.html.erb
@@ -2,6 +2,7 @@
 <html>
   <head>
     <%= stylesheet_link_tag 'media.css' %>
+    <% # sul_icons.css is used for audio icon in the "Media Content" companion window %>
     <%= stylesheet_link_tag 'sul_icons.css' %>
     <%= javascript_importmap_tags 'media' %>
   </head>

diff --git a/app/views/layouts/preview/pdf.html.erb b/app/views/layouts/preview/pdf.html.erb
@@ -2,7 +2,6 @@
 <html>
   <head>
     <%= stylesheet_link_tag 'companion_window.css' %>
-    <%= stylesheet_link_tag 'sul_icons.css' %>
     <%= javascript_importmap_tags 'document' %>
   </head>
   <body>
-Original file line number
+Diff line change
@@ Expand Up / @@ -2,7 +2,6 @@ @@
     <html>
       <head>
         <%= stylesheet_link_tag 'companion_window.css' %>
-        <%= stylesheet_link_tag 'sul_icons.css' %>
         <%= javascript_importmap_tags 'document' %>
       </head>
       <body>
@@ Expand Down @@