aboutsummaryrefslogtreecommitdiffstats
path: root/pydis_site
diff options
context:
space:
mode:
authorGravatar Johannes Christ <[email protected]>2023-05-10 13:54:15 +0200
committerGravatar Johannes Christ <[email protected]>2023-05-10 13:54:17 +0200
commit3cc0dc390bb2635a1192b669f543410d8fd1e59a (patch)
treefa831f73aaecf1595cbb714ec5627337dcdbe5a2 /pydis_site
parentMerge pull request #968 from python-discord/ruff (diff)
Do not duplicate "Edit on GitHub" link for tags
Closes #963.
Diffstat (limited to 'pydis_site')
-rw-r--r--pydis_site/apps/content/tests/test_views.py8
-rw-r--r--pydis_site/templates/content/base.html2
2 files changed, 9 insertions, 1 deletions
diff --git a/pydis_site/apps/content/tests/test_views.py b/pydis_site/apps/content/tests/test_views.py
index e4f898ef..cfc580c0 100644
--- a/pydis_site/apps/content/tests/test_views.py
+++ b/pydis_site/apps/content/tests/test_views.py
@@ -387,6 +387,14 @@ class TagViewTests(django.test.TestCase):
response.context.get("page")
)
+ def test_tags_have_no_edit_on_github_link(self):
+ """Tags should not have the standard edit on GitHub link."""
+ # The standard "Edit on GitHub" link should not be displayed on tags
+ # because they have their own GitHub icon that links there.
+ Tag.objects.create(name="example", body="Joe William Banks", last_commit=self.commit)
+ response = self.client.get("/pages/tags/example/")
+ self.assertNotContains(response, "Edit on GitHub")
+
def test_tag_root_page(self):
"""Test the root tag page which lists all tags."""
Tag.objects.create(name="tag-1", last_commit=self.commit)
diff --git a/pydis_site/templates/content/base.html b/pydis_site/templates/content/base.html
index b04c7efa..bda6d954 100644
--- a/pydis_site/templates/content/base.html
+++ b/pydis_site/templates/content/base.html
@@ -37,7 +37,7 @@
{% include "content/dropdown.html" %}
{% endif %}
{# Edit on GitHub for content articles #}
- {% if page %}
+ {% if page and not tag %}
<div id="edit-on-github">
<a href="{{ request.path | page_src_url }}">
<i class="fa-solid fa-pencil"></i>