From 69c742547b13640b949782e2e3556006e59d50df Mon Sep 17 00:00:00 2001
From: Chukwuma Nwaugha <nwaughac@gmail.com>
Date: Wed, 27 Nov 2024 15:12:22 +0000
Subject: [PATCH] refine sidebar item and item action

---
 app/src/lib/components/SidebarItem.svelte     |  2 +-
 .../lib/components/SidebarItemActions.svelte  | 47 ++++++++++---------
 2 files changed, 26 insertions(+), 23 deletions(-)

diff --git a/app/src/lib/components/SidebarItem.svelte b/app/src/lib/components/SidebarItem.svelte
index 10ce671..bfc0428 100644
--- a/app/src/lib/components/SidebarItem.svelte
+++ b/app/src/lib/components/SidebarItem.svelte
@@ -30,7 +30,7 @@
 <Button
 	{href}
 	variant="ghost"
-	class="relative no-underline group hover:no-underline justify-start py-6 flex items-center px-2 hover:bg-gray-700 bg-gray-900"
+	class="relative no-underline group justify-between hover:no-underline py-6 flex items-center px-2 hover:bg-gray-700 bg-gray-900"
 	data-sveltekit-noscroll
 	on:click
 >
diff --git a/app/src/lib/components/SidebarItemActions.svelte b/app/src/lib/components/SidebarItemActions.svelte
index 9eed8ef..5ac2f45 100644
--- a/app/src/lib/components/SidebarItemActions.svelte
+++ b/app/src/lib/components/SidebarItemActions.svelte
@@ -21,31 +21,34 @@
 	}
 </script>
 
-<div
-	class="absolute rounded-r-md hidden group/action bg-gray-800/80 hover:bg-gray-800 h-full group-hover:flex items-center right-0 p-2"
->
+<div class="hidden h-full z-50 group-hover:flex items-center p-1">
 	<button
-		class="text-gray-400 group-hover/action:text-gray-100"
-		on:click={() => (openDialog = true)}
+		class="text-gray-400 hover:text-red-500"
+		on:click|stopPropagation|preventDefault={() => (openDialog = true)}
 	>
 		<Trash class="w-4 h-4 inline" />
 	</button>
 </div>
 
-<AlertDialog.Root bind:open={openDialog}>
-	<AlertDialog.Trigger>Open</AlertDialog.Trigger>
-	<AlertDialog.Content class="border-neutral-800">
-		<AlertDialog.Header>
-			<AlertDialog.Title>Delete audiocast?</AlertDialog.Title>
-			<AlertDialog.Description>
-				This action cannot be undone. This will permanently delete your audiocast data.
-			</AlertDialog.Description>
-		</AlertDialog.Header>
-		<AlertDialog.Footer>
-			<AlertDialog.Cancel on:click={() => (openDialog = false)}>Cancel</AlertDialog.Cancel>
-			<AlertDialog.Action class="bg-red-500 text-red-100 hover:bg-red-600" on:click={deleteSession}
-				>Continue</AlertDialog.Action
-			>
-		</AlertDialog.Footer>
-	</AlertDialog.Content>
-</AlertDialog.Root>
+{#if openDialog}
+	<AlertDialog.Root open portal="body" closeOnOutsideClick closeOnEscape>
+		<AlertDialog.Content class="border-neutral-800 overflow-hidden">
+			<AlertDialog.Header>
+				<AlertDialog.Title>Delete audiocast?</AlertDialog.Title>
+				<AlertDialog.Description>
+					This action cannot be undone. This will permanently delete your audiocast data.
+				</AlertDialog.Description>
+			</AlertDialog.Header>
+
+			<AlertDialog.Footer>
+				<AlertDialog.Cancel on:click={() => (openDialog = false)}>Cancel</AlertDialog.Cancel>
+				<AlertDialog.Action
+					class="bg-red-700 text-red-100 hover:bg-red-600"
+					on:click={deleteSession}
+				>
+					Continue
+				</AlertDialog.Action>
+			</AlertDialog.Footer>
+		</AlertDialog.Content>
+	</AlertDialog.Root>
+{/if}