From f02edd3699c1f0f82102f33725e7699da1ef6ed6 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Wed, 6 May 2026 20:29:06 +0000 Subject: [PATCH] =?UTF-8?q?=F0=9F=8E=A8=20Palette:=20Add=20accessibility?= =?UTF-8?q?=20attributes=20and=20focus=20rings=20to=20Navbar?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: belpythons <187399139+belpythons@users.noreply.github.com> --- .jules/palette.md | 3 +++ src/components/layout/Navbar.jsx | 15 ++++++++++----- 2 files changed, 13 insertions(+), 5 deletions(-) create mode 100644 .jules/palette.md diff --git a/.jules/palette.md b/.jules/palette.md new file mode 100644 index 0000000..77a34f9 --- /dev/null +++ b/.jules/palette.md @@ -0,0 +1,3 @@ +## 2024-05-06 - Interactive Element Accessibility in Global Navigation +**Learning:** Disclosure widgets (like mobile menus and dropdowns) in global navigation often lack `aria-expanded` attributes, making their state opaque to screen readers. Additionally, icon-only buttons frequently lack explicit focus rings, rendering them inaccessible to keyboard users. +**Action:** Always include `aria-expanded` on any interactive disclosure widget and ensure explicit `focus-visible` styles (e.g., `focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-accent-500`) are applied to all interactive elements, particularly icon-only buttons. diff --git a/src/components/layout/Navbar.jsx b/src/components/layout/Navbar.jsx index de3fea5..3fee993 100644 --- a/src/components/layout/Navbar.jsx +++ b/src/components/layout/Navbar.jsx @@ -95,7 +95,9 @@ const Navbar = () => {
@@ -180,7 +185,7 @@ const Navbar = () => {
{/* Mobile Navigation */} -
+
{navLinks.map((link) => (