Accessibility and style tweaks

fonsp · Sep 19, 2023 · a3ffc09 · a3ffc09
1 parent a7908dd
commit a3ffc09
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 9 deletions.
diff --git a/frontend/components/ExportBanner.js b/frontend/components/ExportBanner.js
@@ -44,7 +44,7 @@ export const WarnForVisisblePasswords = () => {
     }
 }
 
-export const ExportBanner = ({ notebook_id, onClose, notebookfile_url, notebookexport_url, start_recording }) => {
+export const ExportBanner = ({ notebook_id, open, onClose, notebookfile_url, notebookexport_url, start_recording }) => {
     // @ts-ignore
     const isDesktop = !!window.plutoDesktop
 
@@ -56,7 +56,7 @@ export const ExportBanner = ({ notebook_id, onClose, notebookfile_url, notebooke
     }
 
     return html`
-        <aside id="export">
+        <aside id="export" inert=${!open}>
             <div id="container">
                 <div class="export_title">export</div>
                 <!-- no "download" attribute here: we want the jl contents to be shown in a new tab -->

diff --git a/frontend/components/SlideControls.js b/frontend/components/SlideControls.js
@@ -89,7 +89,7 @@ export const SlideControls = () => {
     }, [presenting])
 
     return html`
-        <nav id="slide_controls">
+        <nav id="slide_controls" inert=${!presenting}>
             <button ref=${button_prev_ref} class="changeslide prev" title="Previous slide" onClick=${go_previous_slide}><span></span></button>
             <button ref=${button_next_ref} class="changeslide next" title="Next slide" onClick=${go_next_slide}><span></span></button>
         </nav>

diff --git a/frontend/components/UndoDelete.js b/frontend/components/UndoDelete.js
@@ -24,6 +24,7 @@ export const UndoDelete = ({ recently_deleted, on_click }) => {
     return html`
         <nav
             id="undo_delete"
+            inert=${hidden}
             class=${cl({
                 hidden: hidden,
             })}

diff --git a/frontend/editor.css b/frontend/editor.css
@@ -600,19 +600,20 @@ nav#at_the_top > a {
     /* margin-right: 5rem; */
 }
 
-nav#at_the_top img {
-    height: 39px;
-    margin-bottom: -8px;
-    filter: var(--image-filters);
-}
-
 nav#at_the_top h1 {
     font-weight: 700;
     font-size: 1.8rem;
     display: inline;
     border-bottom: none;
 }
 
+nav#at_the_top img#logo-small,
+nav#at_the_top img#logo-big {
+    height: 39px;
+    margin-bottom: -8px;
+    filter: var(--image-filters);
+}
+
 nav#at_the_top img#logo-small {
     display: none;
 }