From 9ba3a133f1702ec49c6112dfb07dca76bb0e6900 Mon Sep 17 00:00:00 2001
From: Aleksandra Nenadic <a.nenadic@manchester.ac.uk>
Date: Mon, 12 Oct 2026 15:43:01 +0100
Subject: [PATCH 1/2] Render external links with arrow icon and use consistent
 font size in tables

---
 main/tables.py            | 19 +++++++++----------
 tests/main/test_tables.py | 12 +++++++++---
 2 files changed, 18 insertions(+), 13 deletions(-)

diff --git a/main/tables.py b/main/tables.py
index 9f6d2826..931300af 100644
--- a/main/tables.py
+++ b/main/tables.py
@@ -13,10 +13,14 @@
 if TYPE_CHECKING:  # pragma: no cover
     from django.db.models.fields.related_descriptors import ManyRelatedManager
 
+
 external_link_html = (
-    '<a href="{}" target="_blank" rel="noopener noreferrer" class="{}">{}</a>'
+    '<a href="{}" target="_blank" rel="noopener noreferrer" '
+    'class="text-decoration-none fs-xs fw-semibold">'
+    '{}<i class="fa-solid fa-arrow-up-right-from-square ms-1" '
+    'aria-hidden="true"></i></a>'
 )
-badge_html = '<span class="badge bg-secondary fs-sm">{}</span>'
+badge_html = '<span class="badge bg-secondary">{}</span>'
 
 
 def _render_skills(qs: QuerySet[Skill]) -> SafeString:
@@ -78,12 +82,7 @@ def render_url(self, value: str | None) -> SafeString:
         if not value:
             return mark_safe("")
 
-        return format_html(
-            external_link_html,
-            value,
-            "",
-            "Visit website",
-        )
+        return format_html(external_link_html, value, "Visit website")
 
 
 class LearningResourceTable(tables.Table):
@@ -100,7 +99,7 @@ class Meta:
 
     def render_name(self, value: str, record: LearningResource) -> SafeString:
         """Include the URL in the name."""
-        return format_html(external_link_html, record.url, "fs-lg", value)
+        return format_html(external_link_html, record.url, value)
 
     def render_description(self, value: str) -> SafeString:
         """Render the description field."""
@@ -137,7 +136,7 @@ class Meta:
 
     def render_name(self, value: str, record: ToolLanguageMethodology) -> SafeString:
         """Include the URL in the name."""
-        return format_html(external_link_html, record.url, "fs-lg", value)
+        return format_html(external_link_html, record.url, value)
 
     def render_kind(self, value: str) -> SafeString:
         """Render the kind field as a badge."""
diff --git a/tests/main/test_tables.py b/tests/main/test_tables.py
index 4a8e0c1e..40c282d4 100644
--- a/tests/main/test_tables.py
+++ b/tests/main/test_tables.py
@@ -16,7 +16,10 @@ def test_learning_resources_table_render_name(learning_resource: LearningResourc
 
     assert str(rendered) == (
         f'<a href="{learning_resource.url}" target="_blank" '
-        'rel="noopener noreferrer" class="fs-lg">Learning Resource</a>'
+        'rel="noopener noreferrer" '
+        'class="text-decoration-none fs-xs fw-semibold">Learning Resource'
+        '<i class="fa-solid fa-arrow-up-right-from-square ms-1" '
+        'aria-hidden="true"></i></a>'
     )
 
 
@@ -78,7 +81,10 @@ def test_tool_language_methodology_table_render_name(
 
     assert str(rendered) == (
         f'<a href="{tool.url}" target="_blank" '
-        'rel="noopener noreferrer" class="fs-lg">Tool</a>'
+        'rel="noopener noreferrer" '
+        'class="text-decoration-none fs-xs fw-semibold">Tool'
+        '<i class="fa-solid fa-arrow-up-right-from-square ms-1" '
+        'aria-hidden="true"></i></a>'
     )
 
 
@@ -91,7 +97,7 @@ def test_tool_language_methodology_table_render_kind(
 
     rendered = table.render_kind(tool.kind)
 
-    assert str(rendered) == '<span class="badge bg-secondary fs-sm">tool</span>'
+    assert str(rendered) == '<span class="badge bg-secondary">tool</span>'
 
 
 @pytest.mark.django_db

From 973fdf19d4ac59d0893fd1f61021b67fc46bf935 Mon Sep 17 00:00:00 2001
From: Adrian D'Alessandro <a.dalessandro@imperial.ac.uk>
Date: Wed, 14 Oct 2026 12:50:05 +0100
Subject: [PATCH 2/2] Add external link indicator to learning provider and
 skill pages

---
 .../main/pages/learning-provider-detail.html  |  2 ++
 main/templates/main/pages/skill.html          | 31 ++++++++++++++++---
 2 files changed, 28 insertions(+), 5 deletions(-)

diff --git a/main/templates/main/pages/learning-provider-detail.html b/main/templates/main/pages/learning-provider-detail.html
index e3197105..bd38f3fc 100644
--- a/main/templates/main/pages/learning-provider-detail.html
+++ b/main/templates/main/pages/learning-provider-detail.html
@@ -22,6 +22,8 @@ <h1>{{ provider.name }}</h1>
             <p>
                 <a href="{{ provider.url }}" target="_blank" rel="noopener noreferrer">
                     Visit provider website
+                    <i class="fa-solid fa-arrow-up-right-from-square ms-1 fs-xs"
+                       aria-hidden="true"></i>
                 </a>
             </p>
         {% endif %}
diff --git a/main/templates/main/pages/skill.html b/main/templates/main/pages/skill.html
index 6f72f1c8..7dd7ab52 100644
--- a/main/templates/main/pages/skill.html
+++ b/main/templates/main/pages/skill.html
@@ -38,11 +38,20 @@ <h2 class="h4 mt-5">Learning resources</h2>
                                 {% if resource.url %}
                                     <a href="{{ resource.url }}"
                                        target="_blank"
-                                       rel="noopener noreferrer">{{ resource.name }}</a>
+                                       rel="noopener noreferrer">
+                                        {{ resource.name }}
+                                        <i class="fa-solid fa-arrow-up-right-from-square fs-xs"
+                                           aria-hidden="true"></i>
+                                    </a>
                                 {% else %}
                                     {{ resource.name }}
                                 {% endif %}
-                                {% if resource.provider %}- {{ resource.provider.name }}{% endif %}
+                                {% if resource.provider %}
+                                    - Provider:
+                                    <a href="{% url 'learning_provider_detail' resource.provider.slug %}">
+                                        {{ resource.provider.name }}
+                                    </a>
+                                {% endif %}
                             </li>
                         {% endfor %}
                     </ul>
@@ -55,7 +64,11 @@ <h2 class="h4 mt-5">Tools</h2>
                                 {% if item.url %}
                                     <a href="{{ item.url }}"
                                        target="_blank"
-                                       rel="noopener noreferrer">{{ item.name }}</a>
+                                       rel="noopener noreferrer">
+                                        {{ item.name }}
+                                        <i class="fa-solid fa-arrow-up-right-from-square fs-xs"
+                                           aria-hidden="true"></i>
+                                    </a>
                                 {% else %}
                                     {{ item.name }}
                                 {% endif %}
@@ -71,7 +84,11 @@ <h2 class="h4 mt-5">Languages</h2>
                                 {% if item.url %}
                                     <a href="{{ item.url }}"
                                        target="_blank"
-                                       rel="noopener noreferrer">{{ item.name }}</a>
+                                       rel="noopener noreferrer">
+                                        {{ item.name }}
+                                        <i class="fa-solid fa-arrow-up-right-from-square fs-xs"
+                                           aria-hidden="true"></i>
+                                    </a>
                                 {% else %}
                                     {{ item.name }}
                                 {% endif %}
@@ -87,7 +104,11 @@ <h2 class="h4 mt-5">Methodologies</h2>
                                 {% if item.url %}
                                     <a href="{{ item.url }}"
                                        target="_blank"
-                                       rel="noopener noreferrer">{{ item.name }}</a>
+                                       rel="noopener noreferrer">
+                                        {{ item.name }}
+                                        <i class="fa-solid fa-arrow-up-right-from-square fs-xs"
+                                           aria-hidden="true"></i>
+                                    </a>
                                 {% else %}
                                     {{ item.name }}
                                 {% endif %}