From e8098c8e5cf7cde6cd432fb63c2533a1e5737061 Mon Sep 17 00:00:00 2001
From: Suzanne Aitchison <suzanne@forem.com>
Date: Mon, 5 Apr 2021 09:24:56 +0100
Subject: [PATCH] ensure home and sign up links in sidebar are announced once
 to screen reader users (#13231)

---
 app/views/layouts/_main_nav.html.erb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/views/layouts/_main_nav.html.erb b/app/views/layouts/_main_nav.html.erb
index 65896a463..4c7b57c46 100644
--- a/app/views/layouts/_main_nav.html.erb
+++ b/app/views/layouts/_main_nav.html.erb
@@ -2,12 +2,12 @@
 
 <nav class="mb-6 <% unless user_signed_in? %>mt-4<% end %>" id="sidebar-nav" aria-label="Primary sidebar nav">
   <a href="<%= root_path %>" class="crayons-link crayons-link--block">
-    <%= inline_svg_tag("twemoji/house.svg", aria: true, class: "crayons-icon crayons-icon--default", title: "Home") %>
+    <%= inline_svg_tag("twemoji/house.svg", class: "crayons-icon crayons-icon--default", aria_hidden: true) %>
     Home
   </a>
   <% unless user_signed_in? %>
     <a href="<%= sign_up_path %>" class="crayons-link crayons-link--block fw-bold">
-      <%= inline_svg_tag("twemoji/handshake.svg", aria: true, class: "crayons-icon crayons-icon--default", title: "Sign In/Up") %>
+      <%= inline_svg_tag("twemoji/handshake.svg", class: "crayons-icon crayons-icon--default", aria_hidden: true) %>
       Sign In/Up
     </a>
   <% end %>