fix(LandingNav): add a11y role

sabertazimi · Aug 4, 2021 · 5f9f24b · 5f9f24b
1 parent 1cb18ba
commit 5f9f24b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/components/LandingNav/LandingNav.tsx b/src/components/LandingNav/LandingNav.tsx
@@ -28,6 +28,7 @@ const LandingNav = ({ routes }: Props): JSX.Element => {
             '-translate-x-full': !expanded,
           }
         )}
+        role="navigation"
       >
         {routes.map((route) => (
           <LandingNavLink key={route.id} title={route.title} to={route.path}>
@@ -61,6 +62,7 @@ const LandingNav = ({ routes }: Props): JSX.Element => {
             'bg-opacity-0': !expanded,
           }
         )}
+        role="banner"
       />
     </React.Fragment>
   );

diff --git a/src/components/LandingNav/LandingNavLink.tsx b/src/components/LandingNav/LandingNavLink.tsx
@@ -15,7 +15,7 @@ const LandingNavLink = ({
   children,
 }: Props): JSX.Element => (
   <Tooltip className="mb-16" placement="left" title={title}>
-    <Link to={to}>
+    <Link to={to} role="link">
       <span
         className={classNames(
           'block px-0 pt-0 pb-3 mx-6 my-0 border-solid border-b-6 border-light',