removed duplicate h1 + added aria-current

nowaalex · Oct 24, 2023 · 848322e · 848322e
1 parent 8e1d6cd
commit 848322e
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/packages/website/components/Menu/index.tsx b/packages/website/components/Menu/index.tsx
@@ -50,6 +50,7 @@ type MenuProps = JSX.IntrinsicElements["nav"] & {
 const Menu = ({ className, items, prefix, ...props }: MenuProps) => (
     <nav
         {...props}
+        aria-label="Main navigation"
         className={cx(
             "prose prose-sm prose-ul:list-none prose-a:no-underline p-4",
             className

diff --git a/packages/website/components/NavLink/index.tsx b/packages/website/components/NavLink/index.tsx
@@ -27,6 +27,7 @@ const NavLink = ({
         <Link
             {...props}
             href={href}
+            aria-current={isActive ? "page" : undefined}
             className={cx(className, isActive ? activeClassName : "")}
         />
     );

diff --git a/packages/website/components/layouts/Documentation/index.tsx b/packages/website/components/layouts/Documentation/index.tsx
@@ -6,12 +6,17 @@ import Link from "next/link";
 import cx from "utils/cx";
 
 const renderTopLink = (className: string, productName: string) => (
-    <h1 className={cx("flex gap-2 font-medium", className)}>
+    <nav aria-label="Breadcrumb" className={cx("font-medium", className)}>
         <Link href="/" className="underline">
             af-utils
         </Link>
-        /<span className="text-slate-500">{productName}</span>
-    </h1>
+        <span className="px-2" aria-hidden="true">
+            /
+        </span>
+        <span className="text-slate-500" aria-current="page">
+            {productName}
+        </span>
+    </nav>
 );
 
 const getDocumentationLayout =