From 44be86c652a0a754f536bb61143ee91f0942727e Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Tue, 5 May 2026 20:03:33 +0000
Subject: [PATCH] =?UTF-8?q?=F0=9F=8E=A8=20Palette:=20[UX=20improvement]=20?=
 =?UTF-8?q?Enhance=20Navbar=20accessibility?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Added missing accessibility attributes to Navbar interactive elements:
- `aria-expanded` and `aria-haspopup` to the Forum dropdown button
- `aria-label`, `aria-expanded`, and `aria-controls` to the mobile menu toggle button
- `id="mobile-menu"` to the mobile menu container
- `focus-visible` outline rings to both buttons for keyboard navigation

Co-authored-by: belpythons <187399139+belpythons@users.noreply.github.com>
---
 src/components/layout/Navbar.jsx | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/src/components/layout/Navbar.jsx b/src/components/layout/Navbar.jsx
index de3fea5..b5fa93e 100644
--- a/src/components/layout/Navbar.jsx
+++ b/src/components/layout/Navbar.jsx
@@ -95,7 +95,9 @@ const Navbar = () => {
             <div className="relative" ref={dropdownRef}>
               <button
                 onClick={toggleDropdown}
-                className={`flex items-center space-x-1 px-3 py-2 rounded-md text-sm font-medium transition-colors duration-200 ${location.pathname.startsWith('/forum')
+                aria-expanded={isDropdownOpen}
+                aria-haspopup="true"
+                className={`flex items-center space-x-1 px-3 py-2 rounded-md text-sm font-medium transition-colors duration-200 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-accent-500 ${location.pathname.startsWith('/forum')
                     ? 'text-accent-500 bg-accent-50 dark:bg-navy-800'
                     : 'text-gray-700 dark:text-slate-300 hover:text-accent-500 hover:bg-gray-50 dark:hover:bg-navy-800'
                   }`}
@@ -172,7 +174,10 @@ const Navbar = () => {
 
             <button
               onClick={toggleMobile}
-              className="text-gray-700 dark:text-slate-300 hover:text-accent-500 p-2"
+              aria-label={isOpen ? 'Close main menu' : 'Open main menu'}
+              aria-expanded={isOpen}
+              aria-controls="mobile-menu"
+              className="text-gray-700 dark:text-slate-300 hover:text-accent-500 p-2 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-accent-500"
             >
               <FontAwesomeIcon icon={isOpen ? faTimes : faBars} className="text-xl" />
             </button>
@@ -180,7 +185,7 @@ const Navbar = () => {
         </div>
 
         {/* Mobile Navigation */}
-        <div className={`md:hidden transition-all duration-300 ease-in-out ${isOpen ? 'max-h-screen opacity-100' : 'max-h-0 opacity-0 overflow-hidden'}`}>
+        <div id="mobile-menu" className={`md:hidden transition-all duration-300 ease-in-out ${isOpen ? 'max-h-screen opacity-100' : 'max-h-0 opacity-0 overflow-hidden'}`}>
           <div className="px-2 pt-2 pb-3 space-y-1 bg-white dark:bg-navy-900 border-t border-gray-200 dark:border-navy-700">
             {navLinks.map((link) => (
               <Link