From 2d03f1f31bae7367f82e86e8e902bd910e720d27 Mon Sep 17 00:00:00 2001
From: Kenneth <kennethnym@hey.com>
Date: Wed, 21 May 2025 13:47:33 +0100
Subject: [PATCH] implement bookmark link copy

---
 .../web/src/app/bookmarks/-action-bar.tsx     | 21 ++++++++++-----
 .../web/src/app/bookmarks/-bookmark-list.tsx  | 19 ++++++++++++-
 packages/web/src/app/bookmarks/-store.tsx     | 27 +++++++++++++------
 3 files changed, 51 insertions(+), 16 deletions(-)

diff --git a/packages/web/src/app/bookmarks/-action-bar.tsx b/packages/web/src/app/bookmarks/-action-bar.tsx
index dfdb724..5633921 100644
--- a/packages/web/src/app/bookmarks/-action-bar.tsx
+++ b/packages/web/src/app/bookmarks/-action-bar.tsx
@@ -26,6 +26,7 @@ function ActionBar({
 
 function BookmarkListActionBar({ className }: { className?: string }) {
 	const setActiveDialog = useBookmarkPageStore((state) => state.setActiveDialog)
+	const statusMessage = useBookmarkPageStore((state) => state.statusMessage)
 
 	useMnemonics(
 		{
@@ -40,13 +41,19 @@ function BookmarkListActionBar({ className }: { className?: string }) {
 
 	return (
 		<ActionBar className={className}>
-			<Button onClick={addBookmark}>
-				<span className="underline">A</span>DD
-			</Button>
-			<Button>
-				<span className="underline">S</span>EARCH
-			</Button>
-			<LogOutButton />
+			{statusMessage ? (
+				<p>{statusMessage}</p>
+			) : (
+				<>
+					<Button onClick={addBookmark}>
+						<span className="underline">A</span>DD
+					</Button>
+					<Button>
+						<span className="underline">S</span>EARCH
+					</Button>
+					<LogOutButton />
+				</>
+			)}
 		</ActionBar>
 	)
 }
diff --git a/packages/web/src/app/bookmarks/-bookmark-list.tsx b/packages/web/src/app/bookmarks/-bookmark-list.tsx
index 0d56736..99578fc 100644
--- a/packages/web/src/app/bookmarks/-bookmark-list.tsx
+++ b/packages/web/src/app/bookmarks/-bookmark-list.tsx
@@ -15,6 +15,7 @@ enum BookmarkListItemAction {
 	Open = "Open",
 	Edit = "Edit",
 	Delete = "Delete",
+	CopyLink = "CopyLink",
 }
 
 type SelectionChangeCallback = (bookmark: Bookmark) => void
@@ -178,12 +179,24 @@ const _BookmarkList = memo(({ className }: { className?: string }) => {
 			d: deleteItem,
 
 			Enter: openItem,
+
+			c: (event) => {
+				if (event.ctrlKey || event.metaKey) {
+					event.preventDefault()
+					copyBookmarkLink()
+				}
+			},
 		},
 		{
 			ignore: useCallback(() => useBookmarkPageStore.getState().activeDialog !== ActiveDialog.None, []),
 		},
 	)
 
+	async function copyBookmarkLink() {
+		const { bookmarks, selectedIndex, onItemAction } = store.getState()
+		onItemAction(bookmarks[selectedIndex], BookmarkListItemAction.CopyLink)
+	}
+
 	function openItem() {
 		const { bookmarks, selectedIndex, onItemAction } = store.getState()
 		expandItem()
@@ -261,6 +274,10 @@ const BookmarkListItem = memo(
 			onItemAction(bookmark, BookmarkListItemAction.Delete)
 		}
 
+		function copyItemLink() {
+			onItemAction(bookmark, BookmarkListItemAction.CopyLink)
+		}
+
 		return (
 			<li
 				className={clsx("group flex flex-row justify-start py-2", {
@@ -296,7 +313,7 @@ const BookmarkListItem = memo(
 							<BookmarkTagList bookmark={bookmark} />
 							<div className="flex flex-col space-y-1 md:flex-row md:space-y-0 md:space-x-2 items-end justify-between pt-2">
 								<div className="flex space-x-2">
-									<Button variant="light" className="text-sm">
+									<Button variant="light" className="text-sm" onClick={copyItemLink}>
 										<span>COPY LINK</span>
 									</Button>
 									<Button variant="light" className="text-sm">
diff --git a/packages/web/src/app/bookmarks/-store.tsx b/packages/web/src/app/bookmarks/-store.tsx
index 73f83fa..57d6882 100644
--- a/packages/web/src/app/bookmarks/-store.tsx
+++ b/packages/web/src/app/bookmarks/-store.tsx
@@ -14,29 +14,27 @@ enum ActiveDialog {
 	DeleteBookmark = "DeleteBookmark",
 }
 
+const STATUS_MESSAGE_DURATION_MS = 2000
+
 interface BookmarkPageState {
 	bookmarkToBeDeleted: Bookmark | null
 	layoutMode: LayoutMode
 	activeDialog: ActiveDialog
+	statusMessage: string
 
 	handleBookmarkListItemAction: (bookmark: Bookmark, action: BookmarkListItemAction) => void
 	setActiveDialog: (dialog: ActiveDialog) => void
 	setLayoutMode: (mode: LayoutMode) => void
+	showStatus: (message: string) => void
 }
 
 const useBookmarkPageStore = create<BookmarkPageState>()((set, get) => ({
-	bookmarkCount: 0,
-	bookmarks: [],
-	selectedBookmarkId: "",
-	selectedBookmarkIndex: 0,
-	isBookmarkItemExpanded: false,
-	isBookmarkPreviewOpened: false,
 	bookmarkToBeDeleted: null,
 	layoutMode: LayoutMode.Popup,
 	activeDialog: ActiveDialog.None,
-	actionBarHeight: 0,
+	statusMessage: "",
 
-	handleBookmarkListItemAction(bookmark: Bookmark, action: BookmarkListItemAction) {
+	async handleBookmarkListItemAction(bookmark: Bookmark, action: BookmarkListItemAction) {
 		switch (action) {
 			case BookmarkListItemAction.Open:
 				router.navigate({ to: `/bookmarks/${bookmark.id}` })
@@ -45,6 +43,12 @@ const useBookmarkPageStore = create<BookmarkPageState>()((set, get) => ({
 			case BookmarkListItemAction.Delete:
 				set({ bookmarkToBeDeleted: bookmark, activeDialog: ActiveDialog.DeleteBookmark })
 				break
+
+			case BookmarkListItemAction.CopyLink:
+				await navigator.clipboard.writeText(bookmark.url)
+				get().showStatus("Link copied to clipboard!")
+				break
+
 			default:
 				break
 		}
@@ -57,6 +61,13 @@ const useBookmarkPageStore = create<BookmarkPageState>()((set, get) => ({
 	setLayoutMode(mode: LayoutMode) {
 		set({ layoutMode: mode })
 	},
+
+	showStatus(message: string) {
+		set({ statusMessage: message })
+		setTimeout(() => {
+			set({ statusMessage: "" })
+		}, STATUS_MESSAGE_DURATION_MS)
+	},
 }))
 
 export { LayoutMode, ActiveDialog, useBookmarkPageStore }