From 7c1cc61db41f83afef045c230b0eda89550de012 Mon Sep 17 00:00:00 2001
From: Joseph Hale <me@jhale.dev>
Date: Wed, 14 Feb 2024 00:48:51 +0000
Subject: [PATCH] fix: Add accessibility labels to menu/search buttons

On mobile, the menu and search controls collapse behind a button to save space.

Unfortunately, since those buttons don't have accessibility labels, the deaf and blind can't use them effectively via a screen reader. As a transitive impact, search engines like Google penalize Chirpy sites for this accessibility oversight.

This commit adds appropriate accessibility labels to rectify the problem.
---
 _includes/topbar.html | 4 ++--
 1 file(s) changed, 2 insertion(s)(+), 2 deletion(s)(-)

diff --git a/_includes/topbar.html b/_includes/topbar.html
--- a/_includes/topbar.html
+++ b/_includes/topbar.html
@@ -40,7 +40,7 @@
     </nav>
     <!-- endof #breadcrumb -->
 
-    <button type="button" id="sidebar-trigger" class="btn btn-link">
+    <button type="button" id="sidebar-trigger" class="btn btn-link" aria-label="Menu">
       <i class="fas fa-bars fa-fw"></i>
     </button>
 
@@ -55,7 +55,7 @@
       {% endif %}
     </div>
 
-    <button type="button" id="search-trigger" class="btn btn-link">
+    <button type="button" id="search-trigger" class="btn btn-link" aria-label="Search">
       <i class="fas fa-search fa-fw"></i>
     </button>
 

--
tangled.sh