fix other minor warnings

Author: medyo

src/components/BottomNavigation.js

@@ -14,16 +14,19 @@ function BottomNavigation({ selectedCard, setSelectedCard, setShowSettings }) {
           <a
             key={card.name}
             className={
-              'navigationItem ' + (selectedCard && selectedCard.name == card.name ? 'active' : '')
+              'navigationItem ' + (selectedCard && selectedCard.name === card.name ? 'active' : '')
             }
-            href="#"
+            href="/#"
             onClick={(e) => setSelectedCard(card)}>
             {constantCard.icon}
           </a>
         )
       })}
       {
-        <a className={'navigationItem '} href="#" onClick={(e) => setShowSettings((prev) => !prev)}>
+        <a
+          className={'navigationItem '}
+          href="/#"
+          onClick={(e) => setShowSettings((prev) => !prev)}>
           {<AiOutlineMenu />}
         </a>
       }

src/components/CardItemWithActions.js

@@ -35,7 +35,7 @@ export default function CardItemWithActions({ cardItem, item, index, source }) {
   }
   useEffect(() => {
     setIsBookmarked(userBookmarks.some((bm) => bm.source === source && bm.url === item.url))
-  }, [userBookmarks])
+  }, [userBookmarks, source, item])
 
   return (
     <div key={`${source}-${index}`} className="blockRow">

src/features/changelog/components/Changelog.tsx

@@ -63,6 +63,7 @@ export const Changelog = () => {
               <div key={item.name}>
                 <div className="tooltipHeader">
                   <a
+                    href="/#"
                     className="tooltipVersion"
                     onClick={() => window.open(item.html_url, '_blank')}>
                     {item.name}

src/features/settings/components/SettingsModal.tsx

@@ -157,7 +157,7 @@ export const SettingsModal = ({ showSettings, setShowSettings }: SettingsModalPr
             />
             <p className="settingHint">
               Missing language or technology? create an issue{' '}
-              <a href="#" onClick={(e) => window.open(supportLink, '_blank')}>
+              <a href={supportLink} target="_blank" rel="noreferrer">
                 here
               </a>
             </p>
@@ -181,7 +181,7 @@ export const SettingsModal = ({ showSettings, setShowSettings }: SettingsModalPr
             />
             <p className="settingHint">
               Missing a cool data source? create an issue{' '}
-              <a href="#" onClick={(_e) => window.open(supportLink, '_blank')}>
+              <a href={supportLink} target="_blank" rel="noreferrer">
                 here
               </a>
             </p>