refactor: Minor code cleanup and accessibility improvements

Author: suzit-10

frontend/src/components/CustomMenu.js

@@ -1,17 +1,19 @@
 export function CustomMenu({ items, activeMenuItem, onItemClick }: Object) {
-  const linkCombo = 'link mh2 blue-dark ttc pointer hover-black';
+  const linkCombo = 'link mh2 blue-dark ttc pointer hover-black bw0 bg-transparent';
   const isActive = (itemId) => itemId === activeMenuItem;
 
   return (
     <div className="cf mb2 pb3 pt3-ns ph2-m dib flex bg-grey-light">
       {items.map((item) => (
-        <div
+        <button
           key={item.id}
-          className={isActive(item.id) ? `${linkCombo} bb b--blue-dark bw1 pb1` : linkCombo}
+          className={
+            isActive(item.id) ? `${linkCombo} bw1 br-0 bl-0 bt-0 b--blue-dark pb1` : linkCombo
+          }
           onClick={() => onItemClick(item.id)}
         >
           <span className="db dib-ns">{item.label}</span>
-        </div>
+        </button>
       ))}
     </div>
   );

frontend/src/components/teamsAndOrgs/TeamLinkedProjects.js

@@ -24,7 +24,7 @@ export function TeamLinkedProjects({ viewAllEndpoint, border = true, canUserEdit
   const [selectedProjectStatus, setSelectedProjectStatus] = useState('PUBLISHED');
 
   // eslint-disable-next-line no-unused-vars
-  const [projectsError, projectsLoading, projects, refetch] = useFetchWithAbort(
+  const [, , projects, refetch] = useFetchWithAbort(
     `projects/?teamId=${id}&omitMapResults=true&projectStatuses=${selectedProjectStatus}`,
     id,
   );
@@ -47,7 +47,7 @@ export function TeamLinkedProjects({ viewAllEndpoint, border = true, canUserEdit
       </h3>
       <ViewAllLink link={viewAllEndpoint} />
       <div className="pv2">
-        <div style={{ width: '230px' }}>
+        <div style={{ width: '280px' }}>
           <CustomMenu
             items={items}
             activeMenuItem={selectedProjectStatus}
@@ -63,6 +63,7 @@ export function TeamLinkedProjects({ viewAllEndpoint, border = true, canUserEdit
           >
             {projects?.results?.map((card) => (
               <TeamLinkedProjectCard
+                key={card.projectId}
                 projectId={card.projectId}
                 projectName={card.name}
                 unLinkFunc={canUserEditTeam ? unLinkProjectFromTeam : () => {}}