From 7fbcb26c014c206da74ca1d381ee0f5c23087e73 Mon Sep 17 00:00:00 2001
From: Michel Hidalgo <michel@ekumenlabs.com>
Date: Thu, 30 Jan 2020 15:42:19 -0300
Subject: [PATCH 1/3] Disable links to missing API docs.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
---
 _includes/package_links.html   |  9 ++++-----
 _plugins/rosindex_generator.rb | 11 ++++++++++-
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/_includes/package_links.html b/_includes/package_links.html
index b0f6e80..aed5aec 100644
--- a/_includes/package_links.html
+++ b/_includes/package_links.html
@@ -13,11 +13,10 @@
 {% endif %}
 
 <div class="list-group list-group-sm {%if horizontal%}list-group-horizontal list-group-justified{%endif%}">
-  <a class="{{list_group_class}}"
-     target="_blank"
-     href="{{ package.data.docs_uri }}"
-     title="View API documentation on docs.ros.org"
-     {% unless package.snapshot.documented %}disabled{% endunless %}>
+  <a class="{{list_group_class}} {% if package.data.docs_uri == '' %}disabled{% endif %}" target="_blank"
+     {% unless package.data.docs_uri == '' %}href="{{ package.data.docs_uri }}"{% endunless %}
+     {% if package.data.docs_uri == '' %}href="javascript:void(0)"{% endif %}
+     title="View API documentation">
        <span style="margin-right: 5px;" class="glyphicon glyphicon-file"></span>
        {%if hide_link_labels%}<br>{%endif%}
        API Docs
diff --git a/_plugins/rosindex_generator.rb b/_plugins/rosindex_generator.rb
index 2bf6ba3..371657a 100644
--- a/_plugins/rosindex_generator.rb
+++ b/_plugins/rosindex_generator.rb
@@ -14,7 +14,7 @@
 require 'uri'
 require 'set'
 require 'yaml'
-require "net/http"
+require 'net/http'
 require 'thread'
 
 # local libs
@@ -513,6 +513,15 @@ def extract_package(site, distro, repo, snapshot, checkout_path, path, pkg_type,
         docs_uri = "http://docs.ros.org/#{DEFAULT_LANGUAGE_PREFIX}/#{distro}/p/#{package_name}"
       end
 
+      # ensure documentation uri refers to an existing site
+      url = URI(docs_uri)
+      Net::HTTP.start(url.host, url.port) do |http|
+        response = http.head(url.path)
+        if response.code != '200'
+          docs_uri = ''
+        end
+      end
+
       # try to acquire information on the CI status of the package
       ci_data = get_ci_data(distro, package_name, repo.name)
 

From 9d0c592d02ef810480128293c6b917d943236682 Mon Sep 17 00:00:00 2001
From: Michel Hidalgo <michel@ekumenlabs.com>
Date: Fri, 7 Feb 2020 17:54:24 -0300
Subject: [PATCH 2/3] Use 'else' clause instead of separate 'unless'.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
---
 _includes/package_links.html | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/_includes/package_links.html b/_includes/package_links.html
index aed5aec..59492bc 100644
--- a/_includes/package_links.html
+++ b/_includes/package_links.html
@@ -14,8 +14,11 @@
 
 <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"
-     {% unless package.data.docs_uri == '' %}href="{{ package.data.docs_uri }}"{% endunless %}
-     {% if package.data.docs_uri == '' %}href="javascript:void(0)"{% endif %}
+     {% if package.data.docs_uri == '' %}
+     href="javascript:void(0)"
+     {% else %}
+     href="{{ package.data.docs_uri }}"
+     {% endif %}
      title="View API documentation">
        <span style="margin-right: 5px;" class="glyphicon glyphicon-file"></span>
        {%if hide_link_labels%}<br>{%endif%}

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 3/3] 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 59492bc..6475ef8 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 cbb08ca..6637422 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;