From fed8c2e556a0553643144c97069adb97416bce64 Mon Sep 17 00:00:00 2001
From: Amberroseweeks <weeksamberrose@gmail.com>
Date: Wed, 9 Oct 2024 14:44:15 -0400
Subject: [PATCH] adding focus and keyboard accessibility to the cookie consent
 buttons

---
 src/components/CookieConsentBanner.tsx | 31 ++++++++++++++++++++++----
 src/components/Footer.tsx              | 13 ++++++-----
 2 files changed, 34 insertions(+), 10 deletions(-)

diff --git a/src/components/CookieConsentBanner.tsx b/src/components/CookieConsentBanner.tsx
index 767d1676..b620a852 100644
--- a/src/components/CookieConsentBanner.tsx
+++ b/src/components/CookieConsentBanner.tsx
@@ -3,12 +3,15 @@
 import { ThemeButton } from "./ThemeButton";
 import { Check, X } from "@phosphor-icons/react";
 import { useCookieContext } from "@/context/CookieContext";
-import { useEffect, useState } from "react";
+import { useEffect, useState, useRef } from "react";
 
 const CookieConsentBanner = () => {
   const { shouldShowBanner, setShouldAllowCookies, setShouldShowBanner } =
     useCookieContext();
 
+  const bannerRef = useRef<HTMLDivElement>(null);
+  const inputRef = useRef<HTMLInputElement>(null);
+
   const [isClientSide, setIsClientSide] = useState(false);
 
   useEffect(() => {
@@ -21,10 +24,24 @@ const CookieConsentBanner = () => {
     setShouldShowBanner(false);
   };
 
+  useEffect(() => {
+    // Focus on the banner when it becomes visible
+    if (shouldShowBanner && bannerRef.current) {
+      // Use setTimeout to ensure focus is applied after DOM updates
+      setTimeout(() => {
+        bannerRef.current?.focus();
+      }, 0);
+    }
+  }, [shouldShowBanner]); // Trigger focus when shouldShowBanner changes
+
   if (!isClientSide) return null;
 
   return (
     <div
+      role="region"
+      tabIndex={-1} // Make the div focusable
+      ref={bannerRef} // Attach the ref to the banner
+      aria-labelledby="cookie_heading"
       className={`${
         shouldShowBanner
           ? "md:flex fixed inset-x-0 bottom-0 z-20 justify-between bg-blue-200 p-4 sm:p-6"
@@ -32,16 +49,21 @@ const CookieConsentBanner = () => {
       }`}
     >
       <div>
-        <div className="heading-md">Can we store cookies to your browser?</div>
+        <h2 className="heading-md" id="cookie_heading">
+          Can we store cookies to your browser?
+        </h2>
         <div className="body-sm">
-          This provides you a nice experience preserving your filtering, your
-          position on the map and your property saved list.
+          <p>
+            This provides you a nice experience preserving your filtering, your
+            position on the map and your property saved list.
+          </p>
         </div>
       </div>
 
       <div className="flex justify-end gap-2 pt-4 sm:pt-0">
         <div className="flex flex-none items-center gap-x-2">
           <ThemeButton
+            tabIndex={1}
             color="tertiary"
             label="Decline"
             startContent={<X />}
@@ -49,6 +71,7 @@ const CookieConsentBanner = () => {
           />
 
           <ThemeButton
+            tabIndex={2}
             color="primary"
             label="Accept"
             startContent={<Check />}
diff --git a/src/components/Footer.tsx b/src/components/Footer.tsx
index 9a0ab52c..ba2fa799 100644
--- a/src/components/Footer.tsx
+++ b/src/components/Footer.tsx
@@ -2,6 +2,7 @@
 
 import { useCookieContext } from "@/context/CookieContext";
 import CookieConsentBanner from "./CookieConsentBanner";
+import { ThemeButton } from "./ThemeButton";
 import Link from "next/link";
 
 const Footer = () => {
@@ -25,12 +26,12 @@ const Footer = () => {
             <span className="max-sm:hidden">—</span>
 
             <li className="base-sm underline text-gray-600 mx-auto">
-              <a
-                className="hover:text-gray-800 cursor-pointer"
-                onClick={onClickCookieSettings}
-              >
-                Cookie Settings
-              </a>
+              <ThemeButton
+                color="tertiary"
+                label="Cookie Settings"
+                onPress={onClickCookieSettings}
+                className="hover:text-gray-800 cursor-pointer text-gray-600"
+              />
             </li>
 
             <span className="max-sm:hidden">—</span>