fix: lint errors

EddieHubCommunity · Jul 12, 2024 · f6368b3 · f6368b3
1 parent 01ef604
commit f6368b3
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 6 deletions.
diff --git a/src/components/Header.js b/src/components/Header.js
@@ -110,10 +110,12 @@ export default function Header({ session, user }) {
                   <MenuButton className="relative flex rounded-full bg-gray-800 text-sm text-white focus:outline-none focus:ring-2 focus:ring-white focus:ring-offset-2 focus:ring-offset-gray-800">
                     <span className="absolute -inset-1.5" />
                     <span className="sr-only">Open user menu</span>
-                    <img
-                      alt=""
+                    <Image
+                      alt="Profile picture of logged in user"
                       src={user.imageUrl}
                       className="h-8 w-8 rounded-full"
+                      height={32}
+                      width={32}
                     />
                   </MenuButton>
                 </div>
@@ -181,10 +183,12 @@ export default function Header({ session, user }) {
           <div className="border-t border-gray-700 pb-3 pt-4">
             <div className="flex items-center px-4">
               <div className="flex-shrink-0">
-                <img
-                  alt=""
+                <Image
+                  alt="Profile picture of logged in user"
                   src={user.imageUrl}
                   className="h-10 w-10 rounded-full"
+                  height={48}
+                  width={48}
                 />
               </div>
               <div className="ml-3">

diff --git a/src/components/Heading.js b/src/components/Heading.js
@@ -10,7 +10,10 @@ export default function Heading({ title, actions = [], extras = [] }) {
         {extras.length > 0 && (
           <div className="mt-1 flex flex-col sm:mt-0 sm:flex-row sm:flex-wrap sm:space-x-6">
             {extras.map((extra) => (
-              <div className="mt-2 flex items-center text-sm text-gray-300">
+              <div
+                className="mt-2 flex items-center text-sm text-gray-300"
+                key={extra.icon}
+              >
                 <extra.icon
                   aria-hidden="true"
                   className="mr-1.5 h-5 w-5 flex-shrink-0 text-gray-500"
@@ -24,7 +27,7 @@ export default function Heading({ title, actions = [], extras = [] }) {
       {actions.length > 0 && (
         <div className="mt-5 flex lg:ml-4 lg:mt-0">
           {actions.map((action) => (
-            <span className="sm:ml-3">
+            <span className="sm:ml-3" key={action.icon}>
               <Link
                 href={action.url}
                 className="inline-flex items-center rounded-md bg-indigo-500 px-3 py-2 text-sm font-semibold text-white shadow-sm hover:bg-indigo-400 focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2 focus-visible:outline-indigo-500"