From 02cc6143655b2fca60c4b3475a885dbcda4995e8 Mon Sep 17 00:00:00 2001
From: Tully Foote <tullyfoote@intrinsic.ai>
Date: Tue, 2 Jan 2024 01:19:26 -0800
Subject: [PATCH] switch from an <a> to a <div> to disable if it doesn't exist

Fix the formatting lost for the horizontal list that was relying on the a member.
---
 _includes/package_links.html | 17 +++++++++++------
 _sass/_base.scss             |  2 +-
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/_includes/package_links.html b/_includes/package_links.html
index 59492bc2..6475ef8b 100644
--- a/_includes/package_links.html
+++ b/_includes/package_links.html
@@ -13,17 +13,22 @@
 {% endif %}
 
 <div class="list-group list-group-sm {%if horizontal%}list-group-horizontal list-group-justified{%endif%}">
-  <a class="{{list_group_class}} {% if package.data.docs_uri == '' %}disabled{% endif %}" target="_blank"
-     {% if package.data.docs_uri == '' %}
-     href="javascript:void(0)"
-     {% else %}
-     href="{{ package.data.docs_uri }}"
-     {% endif %}
+  {% if package.data.docs_uri == '' %}
+  <div class="{{list_group_class}} disabled" 
      title="View API documentation">
        <span style="margin-right: 5px;" class="glyphicon glyphicon-file"></span>
        {%if hide_link_labels%}<br>{%endif%}
        API Docs
+  </div>
+  {% else %} 
+  <a class="{{list_group_class}}" target="_blank"
+    href="{{ package.data.docs_uri }}"
+    title="View API documentation">
+      <span style="margin-right: 5px;" class="glyphicon glyphicon-file"></span>
+      {%if hide_link_labels%}<br>{%endif%}
+      API Docs
   </a>
+{% endif %}
   <a class="{{list_group_class}}"
      target="_blank"
      href="{{ package.data.browse_uri }}"
diff --git a/_sass/_base.scss b/_sass/_base.scss
index cbb08ca5..66374227 100644
--- a/_sass/_base.scss
+++ b/_sass/_base.scss
@@ -401,7 +401,7 @@ th.rotate45 {
 }
 
 .list-group-horizontal .list-group-item {
-  display: inline-block;
+  display: table-cell;
 }
 .list-group-horizontal .list-group-item {
   margin-bottom: 0;