fix: add aria-label prop on nav links

Author: snitin315

src/components/Navigation/Navigation.jsx

@@ -22,9 +22,10 @@ NavigationItem.propTypes = {
   children: PropTypes.node.isRequired,
   url: PropTypes.string.isRequired,
   isactive: PropTypes.func,
+  ariaLabel: PropTypes.string,
 };
 
-function NavigationItem({ children, url, isactive }) {
+function NavigationItem({ children, url, isactive, ariaLabel }) {
   let obj = {};
   // decide if the link is active or not by providing a function
   // otherwise we'll let react-dom makes the decision for us
@@ -42,6 +43,7 @@ function NavigationItem({ children, url, isactive }) {
         target="_blank"
         rel="noopener noreferrer"
         className={classes}
+        aria-label={ariaLabel}
       >
         {children}
       </a>
@@ -54,6 +56,7 @@ function NavigationItem({ children, url, isactive }) {
         isActive ? `${classes} !text-blue-200` : classes
       }
       to={url}
+      aria-label={ariaLabel}
     >
       {children}
     </NavLink>
@@ -119,8 +122,13 @@ function Navigation({ links, pathname, hash = '', toggleSidebar }) {
             <Logo />
           </Link>
           <nav className="hidden md:inline-grid md:grid-flow-col md:gap-x-[18px]">
-            {links.map(({ content, url, isActive }) => (
-              <NavigationItem key={url} url={url} isActive={isActive}>
+            {links.map(({ content, url, isActive, ariaLabel }) => (
+              <NavigationItem
+                key={url}
+                url={url}
+                isActive={isActive}
+                ariaLabel={ariaLabel}
+              >
                 {content}
               </NavigationItem>
             ))}

src/components/Site/Site.jsx

@@ -224,6 +224,7 @@ function Site(props) {
           links={[
             {
               content: 'Documentation',
+              ariaLabel: 'webpack documentation',
               url: '/concepts/',
               isactive: (_, location) => {
                 return /^\/(api|concepts|configuration|guides|loaders|migrate|plugins)/.test(
@@ -236,8 +237,12 @@ function Site(props) {
                 )
               ),
             },
-            { content: 'Contribute', url: '/contribute/' },
-            { content: 'Blog', url: '/blog/' },
+            {
+              content: 'Contribute',
+              url: '/contribute/',
+              ariaLabel: 'contribute to webpack',
+            },
+            { content: 'Blog', url: '/blog/', ariaLabel: 'webpack blog' },
           ]}
         />
       </div>