chore: use request.user_agent.is_mobile instead of parameter

Sefaria · Jan 21, 2025 · c59bb70 · c59bb70
1 parent 700ee8f
commit c59bb70
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/templates/_sidebar.html b/templates/_sidebar.html
@@ -1,7 +1,7 @@
 {% load i18n %}
 
     <nav id="aboutSidebar">
-        {% if mobile %}
+        {% if request.user_agent.is_mobile %}
             <h1 class="int-en">{{title}}</h1>
             <h1 class="int-he">{{heTitle}}</h1>
         {% endif %}
@@ -42,7 +42,7 @@ <h1 class="int-he">{{heTitle}}</h1>
         </ul>
     </nav>
 <script>
-  const mobile = "{{mobile}}" === "1";
+  const mobile = "{{request.user_agent.is_mobile}}" === "True";
   if (!mobile) {
       const whichPage = "{{whichPage}}";
       const item = document.getElementById(whichPage);

diff --git a/templates/static/mobile-about-menu.html b/templates/static/mobile-about-menu.html
@@ -5,7 +5,7 @@
 {% block content %}
     <link rel="stylesheet" href="{% static 'css/about.css' %}">
     <div class="mobileAboutPage">
-        {% include '_sidebar.html' with whichPage='' title="About Sefaria" heTitle="אודות ספריא" mobile="1" %}
+        {% include '_sidebar.html' with whichPage='' title="About Sefaria" heTitle="אודות ספריא" %}
     </div>
 {% endblock %}
 {% block footer %} {% endblock %}