From 442a76a57ed6639ee51babe1f7a2616d502e15e1 Mon Sep 17 00:00:00 2001
From: shamoon <4887959+shamoon@users.noreply.github.com>
Date: Mon, 1 May 2023 19:54:33 -0700
Subject: [PATCH] Handle duplicate keys for bookmarks

Closes #1445

Co-Authored-By: David <davidsmejia@gmail.com>
---
 src/components/bookmarks/list.jsx | 2 +-
 src/components/quicklaunch.jsx    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/components/bookmarks/list.jsx b/src/components/bookmarks/list.jsx
index 3b3774c95..d37b83e79 100644
--- a/src/components/bookmarks/list.jsx
+++ b/src/components/bookmarks/list.jsx
@@ -4,7 +4,7 @@ export default function List({ bookmarks }) {
   return (
     <ul className="mt-3 flex flex-col">
       {bookmarks.map((bookmark) => (
-        <Item key={bookmark.name} bookmark={bookmark} />
+        <Item key={`${bookmark.name}-${bookmark.href}`} bookmark={bookmark} />
       ))}
     </ul>
   );
diff --git a/src/components/quicklaunch.jsx b/src/components/quicklaunch.jsx
index f71da9a08..229b2f4d4 100644
--- a/src/components/quicklaunch.jsx
+++ b/src/components/quicklaunch.jsx
@@ -161,7 +161,7 @@ export default function QuickLaunch({servicesAndBookmarks, searchString, setSear
               )} type="text" autoCorrect="false" ref={searchField} value={searchString} onChange={handleSearchChange} onKeyDown={handleSearchKeyDown} />
             {results.length > 0 && <ul className="max-h-[60vh] overflow-y-auto m-2">
               {results.map((r, i) => (
-                <li key={r.container ?? r.app ?? r.name}>
+                <li key={r.container ?? r.app ?? `${r.name}-${r.href}`}>
                   <button type="button" data-index={i} onMouseEnter={handleItemHover} className={classNames(
                     "flex flex-row w-full items-center justify-between rounded-md text-sm md:text-xl py-2 px-4 cursor-pointer text-theme-700 dark:text-theme-200",
                     i === currentItemIndex && "bg-theme-300/50 dark:bg-theme-700/50",