From 4b8c9da7e8fbc8f63174f063ecd64d01f776baf8 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 28 Oct 2026 08:05:09 +0000 Subject: [PATCH 1/5] Add Get help navigation link Co-authored-by: m-aciek <9288014+m-aciek@users.noreply.github.com> --- templates/base.html.jinja | 20 ++++++++++++++++++++ tests/test_index.py | 9 +++++++++ 2 files changed, 29 insertions(+) diff --git a/templates/base.html.jinja b/templates/base.html.jinja index 5ab887e04..c23877466 100644 --- a/templates/base.html.jinja +++ b/templates/base.html.jinja @@ -88,6 +88,26 @@ Translating +
  • + + + Get help + +
    • ', + index, + ) + self.assertIn(' Get help\n', index) + self.assertIn('class="bi bi-box-arrow-up-right"', index) def test_orders_by_completion_score(self): env = Environment(loader=FileSystemLoader('templates')) From 93c0644ea1157e36f4d87e87b5b98f4acf01e7c2 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 28 Oct 2026 10:16:06 +0000 Subject: [PATCH 2/5] Move external-link icons after labels Co-authored-by: m-aciek <9288014+m-aciek@users.noreply.github.com> --- templates/base.html.jinja | 12 ++++++------ tests/test_index.py | 18 ++++++++++++++++-- 2 files changed, 22 insertions(+), 8 deletions(-) diff --git a/templates/base.html.jinja b/templates/base.html.jinja index c23877466..fa93430ed 100644 --- a/templates/base.html.jinja +++ b/templates/base.html.jinja @@ -74,9 +74,10 @@ + Translating @@ -85,7 +86,6 @@ - Translating
  • @@ -93,9 +93,10 @@ href="https://discuss.python.org/c/documentation/translations/" target="_blank" rel="noopener noreferrer"> + Get help - Get help
    • diff --git a/tests/test_index.py b/tests/test_index.py index dfa95230e..1d6a78d44 100644 --- a/tests/test_index.py +++ b/tests/test_index.py @@ -42,8 +42,22 @@ def test_renders(self): ' rel="noopener noreferrer">', index, ) - self.assertIn(' Get help\n', index) - self.assertIn('class="bi bi-box-arrow-up-right"', index) + for href, label in ( + ( + 'https://devguide.python.org/documentation/translations/translating/', + 'Translating', + ), + ( + 'https://discuss.python.org/c/documentation/translations/', + 'Get help', + ), + ): + link = index[index.index(href):index.index('', index.index(href))] + self.assertLess( + link.index(label), link.index('class="bi bi-box-arrow-up-right"') + ) + self.assertIn('width="14"', link) + self.assertIn('height="14"', link) def test_orders_by_completion_score(self): env = Environment(loader=FileSystemLoader('templates')) From 801aac44f5adb2593ac716609fd49c0b0cd1b723 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 28 Oct 2026 16:13:09 +0000 Subject: [PATCH 3/5] Shrink external-link icons and format test Co-authored-by: m-aciek <9288014+m-aciek@users.noreply.github.com> --- templates/base.html.jinja | 8 ++++---- tests/test_index.py | 24 ++++++++++++++---------- 2 files changed, 18 insertions(+), 14 deletions(-) diff --git a/templates/base.html.jinja b/templates/base.html.jinja index fa93430ed..d01a89cbe 100644 --- a/templates/base.html.jinja +++ b/templates/base.html.jinja @@ -76,8 +76,8 @@ target="_blank"> Translating @@ -95,8 +95,8 @@ rel="noopener noreferrer"> Get help ', - index, - ) + + def link_for(href): + link_start = index.index(href) + link_end = index.index('', link_start) + return index[link_start:link_end] + for href, label in ( ( 'https://devguide.python.org/documentation/translations/translating/', @@ -52,12 +51,17 @@ def test_renders(self): 'Get help', ), ): - link = index[index.index(href):index.index('', index.index(href))] + link = link_for(href) + self.assertIn('target="_blank"', link) self.assertLess( link.index(label), link.index('class="bi bi-box-arrow-up-right"') ) - self.assertIn('width="14"', link) - self.assertIn('height="14"', link) + self.assertIn('width="12"', link) + self.assertIn('height="12"', link) + self.assertIn( + 'rel="noopener noreferrer"', + link_for('https://discuss.python.org/c/documentation/translations/'), + ) def test_orders_by_completion_score(self): env = Environment(loader=FileSystemLoader('templates')) From 0cf6fa5fb552a25195e481bb688e65251f0bcedc Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 29 Oct 2026 13:12:29 +0000 Subject: [PATCH 4/5] Update navigation external-link icons Co-authored-by: m-aciek <9288014+m-aciek@users.noreply.github.com> --- src/style.css | 7 +++++++ templates/base.html.jinja | 44 +++++++++++++++------------------------ tests/test_index.py | 10 ++++++--- 3 files changed, 31 insertions(+), 30 deletions(-) diff --git a/src/style.css b/src/style.css index 413f279fa..a05a92f9b 100644 --- a/src/style.css +++ b/src/style.css @@ -96,6 +96,13 @@ body { margin-top: -4.5px; } +.nav-link .external-link-icon { + width: 0.75em; + height: 0.75em; + margin: 0 0 0 0.15em; + vertical-align: baseline; +} + .theme-toggle-btn { border: none; color: var(--navbar-link-color); diff --git a/templates/base.html.jinja b/templates/base.html.jinja index d01a89cbe..53a3f746c 100644 --- a/templates/base.html.jinja +++ b/templates/base.html.jinja @@ -56,17 +56,6 @@