From 8c72558ca69a237e36ffff27e1d02ce3bfc0bb51 Mon Sep 17 00:00:00 2001
From: Jake Howard <git@theorangeone.net>
Date: Wed, 17 Jan 2024 19:35:13 +0000
Subject: [PATCH] Add alt text for unsplash image URLs

---
 website/common/models.py                          | 7 +++++++
 website/common/templates/common/listing-item.html | 2 +-
 website/common/templates/wagtail_base.html        | 2 +-
 3 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/website/common/models.py b/website/common/models.py
index fb9bedd..ad96dbd 100644
--- a/website/common/models.py
+++ b/website/common/models.py
@@ -195,6 +195,13 @@ class BaseContentPage(BasePage, MetadataMixin):
     def list_image_url(self) -> Optional[str]:
         return self.hero_url("small")
 
+    @cached_property
+    def hero_image_alt(self) -> str:
+        if self.hero_unsplash_photo_id is None:
+            return ""
+
+        return self.hero_unsplash_photo.data.get("description", "")
+
     def get_meta_url(self) -> str:
         return self.full_url
 
diff --git a/website/common/templates/common/listing-item.html b/website/common/templates/common/listing-item.html
index 0321cb9..92fedea 100644
--- a/website/common/templates/common/listing-item.html
+++ b/website/common/templates/common/listing-item.html
@@ -6,7 +6,7 @@
     <figure class="media-left column is-3 image-column">
       {% if page.list_image_url %}
         <a href="{% pageurl page %}" class="image" title="{{ page.title }}">
-          <img src="{{ page.list_image_url }}" alt="" loading="lazy" decoding="async" />
+          <img src="{{ page.list_image_url }}" alt="{{ page.hero_image_alt }}" loading="lazy" decoding="async" />
         </a>
       {% endif %}
     </figure>
diff --git a/website/common/templates/wagtail_base.html b/website/common/templates/wagtail_base.html
index 9dfb407..4d7bb69 100644
--- a/website/common/templates/wagtail_base.html
+++ b/website/common/templates/wagtail_base.html
@@ -15,7 +15,7 @@
     <picture>
       {% for width, image_url in page.hero_image_urls.items reversed %}<source srcset="{{ image_url }}" media="(max-width: {{ width }}px)" />{% endfor %}
 
-      <img class="hero" src="{{ page.hero_image_url }}" decoding="async" alt="" />
+      <img class="hero" src="{{ page.hero_image_url }}" decoding="async" alt="{{ page.hero_image_alt }}" />
     </picture>
   {% endif %}