From 71fc4b8da32e3ed5c40ab2a35e31606fe0527f7c Mon Sep 17 00:00:00 2001
From: William Palin <bill@free.law>
Date: Tue, 3 Dec 2024 09:55:45 -0500
Subject: [PATCH 1/2] fix(opinions.html): Enable Harvard SCAN tab

---
 cl/opinion_page/templates/opinions.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/cl/opinion_page/templates/opinions.html b/cl/opinion_page/templates/opinions.html
index 320dbb40d9..37343d034a 100644
--- a/cl/opinion_page/templates/opinions.html
+++ b/cl/opinion_page/templates/opinions.html
@@ -333,10 +333,10 @@ <h4 class="case-court">{{ cluster.docket.court }}</h4>
                         {% endif %}
                     </li>
                 {% endif %}
-                {% if has_downloads and "pdf" in pdf_path %}
+                {% if has_downloads and "pdf" in pdf_path or cluster.filepath_pdf_harvard %}
                     <li role="presentation" {% if tab == "pdf" %} class="active" {% endif %}>
                         {% if tab == "pdf" %}
-                            <a href="#" data-toggle="tab">{% if "pdf" in pdf_path %}PDF{% endif %}</a>
+                            <a href="#" data-toggle="tab">PDF</a>
                         {% else %}
                             <a href="{% url 'view_case_pdf' cluster.pk cluster.slug %}">PDF</a>
                         {% endif %}

From 83b9575711a34763611534e06a5918a7682fea0c Mon Sep 17 00:00:00 2001
From: William Palin <bill@free.law>
Date: Tue, 3 Dec 2024 10:18:49 -0500
Subject: [PATCH 2/2] fix(opinions.html): Remove misc. if statements

Tabs can just link all the time - no need
to hide them
---
 cl/opinion_page/templates/opinions.html | 40 +++++--------------------
 1 file changed, 8 insertions(+), 32 deletions(-)

diff --git a/cl/opinion_page/templates/opinions.html b/cl/opinion_page/templates/opinions.html
index 8d26b09010..6adb8d5cbf 100644
--- a/cl/opinion_page/templates/opinions.html
+++ b/cl/opinion_page/templates/opinions.html
@@ -290,55 +290,31 @@ <h4 class="case-court">{{ cluster.docket.court }}</h4>
 
             <ul class="nav nav-tabs hidden-print">
                 <li role="presentation" {% if tab == "opinions" or tab == "" %} class="active" {% endif %}>
-                    {% if tab == "opinions" %}
-                        <a href="#" data-toggle="tab">Opinion</a>
-                    {% else %}
-                        <a href="{% url 'view_case' cluster.pk cluster.slug %}">Opinion</a>
-                    {% endif %}
+                    <a href="{% url 'view_case' cluster.pk cluster.slug %}">Opinion</a>
                 </li>
                 {% if authorities_count > 0 %}
                     <li role="presentation" {% if tab == "authorities" %} class="active" {% endif %}>
-                        {% if tab == "authorities" %}
-                            <a href="#" data-toggle="tab">Auth<span class="hidden-sm hidden-md hidden-xs">orities&nbsp;({{ authorities_count }})</a>
-                        {% else %}
-                            <a href="{% url "view_case_authorities" cluster.pk cluster.slug %}">Auth<span class="hidden-xs hidden-sm hidden-md">orities&nbsp;({{ authorities_count }})</span></a>
-                        {% endif %}
+                        <a href="{% url "view_case_authorities" cluster.pk cluster.slug %}">Auth<span class="hidden-xs hidden-sm hidden-md">orities&nbsp;({{ authorities_count }})</span></a>
                     </li>
                 {% endif %}
                 {% if cited_by_count > 0 %}
-                <li role="presentation" {% if tab == "cited-by" %} class="active" {% endif %}>
-                    {% if tab == "cited-by" %}
-                        <a href="#" data-toggle="tab">Cited<span class="hidden-xs hidden-sm hidden-md">&nbsp;By&nbsp;({{ cited_by_count }})</span></a>
-                    {% else %}
+                    <li role="presentation" {% if tab == "cited-by" %} class="active" {% endif %}>
                         <a href="{% url "view_case_cited_by" cluster.pk cluster.slug %}">Cited<span class="hidden-xs hidden-sm hidden-md">&nbsp;By&nbsp;({{ cited_by_count }})</span></a>
-                    {% endif %}
-                </li>
+                    </li>
                 {% endif %}
                 {% if summaries_count > 0 %}
-                  <li role="presentation" {% if tab == "summaries" %} class="active" {% endif %}>
-                    {% if tab == "summaries" %}
-                        <a href="#" data-toggle="tab">Sum<span class="hidden-xs hidden-sm hidden-md">maries&nbsp;({{ summaries_count }})</span></a>
-                    {% else %}
+                    <li role="presentation" {% if tab == "summaries" %} class="active" {% endif %}>
                         <a href="{% url "view_case_summaries" cluster.pk cluster.slug %}">Sum<span class="hidden-xs hidden-sm hidden-md">maries&nbsp;({{ summaries_count }})</span></a>
-                    {% endif %}
-                  </li>
+                    </li>
                 {% endif %}
                 {% if related_cases_count > 0  %}
                     <li role="presentation" {% if tab == "related-cases" %} class="active" {% endif %}>
-                        {% if tab == "related-cases" %}
-                            <a href="#" data-toggle="tab">Related<span class="hidden-xs hidden-sm hidden-md">&nbsp;Cases</span></a>
-                        {% else %}
-                            <a href="{% url 'view_case_related_cases' cluster.pk cluster.slug %}">Related&nbsp;<span class="hidden-xs hidden-sm hidden-md">Cases</span></a>
-                        {% endif %}
+                        <a href="{% url 'view_case_related_cases' cluster.pk cluster.slug %}">Related&nbsp;<span class="hidden-xs hidden-sm hidden-md">Cases</span></a>
                     </li>
                 {% endif %}
                 {% if has_downloads and "pdf" in pdf_path or cluster.filepath_pdf_harvard %}
                     <li role="presentation" {% if tab == "pdf" %} class="active" {% endif %}>
-                        {% if tab == "pdf" %}
-                            <a href="#" data-toggle="tab">PDF</a>
-                        {% else %}
-                            <a href="{% url 'view_case_pdf' cluster.pk cluster.slug %}">PDF</a>
-                        {% endif %}
+                        <a href="{% url 'view_case_pdf' cluster.pk cluster.slug %}">PDF</a>
                     </li>
                 {% endif %}
             </ul>