chore: make mobile about menu case the only time header appears makin…

…g css display none unnecessary
Sefaria · Jan 20, 2025 · 700ee8f · 700ee8f
1 parent 073b5ab
commit 700ee8f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 8 deletions.
diff --git a/static/css/static.css b/static/css/static.css
@@ -3520,13 +3520,8 @@ form.globalUpdateForm + div.notificationsList {
       margin-right: 20px;
     }
 }
-nav#aboutSidebar h1 {
-  display: none;
-}
+
 @media (max-width: 480px) {
-  .interface-english nav#aboutSidebar h1.int-en, .interface-hebrew nav#aboutSidebar h1.int-he {
-    display: block;
-  }
   .container.doc.legal.static .inner, #annualReport .inner, #metricsPage .inner, #supportersPage .inner,
   #testimonialsPage .inner, #aboutPage .inner, #teamPage .inner, #jobsPage .inner {
     margin-inline-start: auto;

diff --git a/templates/_sidebar.html b/templates/_sidebar.html
@@ -1,8 +1,10 @@
 {% load i18n %}
 
     <nav id="aboutSidebar">
-        <h1 class="int-en">{{title}}</h1>
-        <h1 class="int-he">{{heTitle}}</h1>
+        {% if mobile %}
+            <h1 class="int-en">{{title}}</h1>
+            <h1 class="int-he">{{heTitle}}</h1>
+        {% endif %}
         <ul>
             <li><a id="about" href="/about">
                     <span class="int-en">What is Sefaria?<i class="fa fa-chevron-right"></i></span>