From edaa2d2de8f92d80f1c61f5a62b295e708219355 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Javi=20Mart=C3=ADn?= <javim@elretirao.net>
Date: Sat, 12 Jun 2021 00:13:45 +0200
Subject: [PATCH] Fix missing alt attribute in phases images

An image without an alt text is invalid HTML and is confusing for screen
reader users.

This is just a quick patch which partially solves this problem. The
image doesn't necessarily need to be a decorative one, so administrators
should have the option to provide an alternative text for the images.
---
 app/components/budgets/phases_component.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/components/budgets/phases_component.html.erb b/app/components/budgets/phases_component.html.erb
index 6e4b66bf0..8c881a4f1 100644
--- a/app/components/budgets/phases_component.html.erb
+++ b/app/components/budgets/phases_component.html.erb
@@ -55,7 +55,7 @@
 
           <% if phase.image.present? %>
             <div class="budget-phase-image">
-              <%= image_tag phase.image.attachment.url(:large) %>
+              <%= image_tag phase.image.attachment.url(:large), alt: "" %>
             </div>
           <% end %>
         </div>