From 1be2268e547a1ae8c1256568dd98657d00200f28 Mon Sep 17 00:00:00 2001
From: Josh Creek <8179928+jcreek@users.noreply.github.com>
Date: Wed, 16 Apr 2025 23:54:14 +0100
Subject: [PATCH] feat(*): Add confirmation dialogs

---
 src/routes/results/+page.svelte | 33 +++++++++++++++++++--------------
 1 file changed, 19 insertions(+), 14 deletions(-)

diff --git a/src/routes/results/+page.svelte b/src/routes/results/+page.svelte
index 92e2d68..d499b79 100644
--- a/src/routes/results/+page.svelte
+++ b/src/routes/results/+page.svelte
@@ -42,26 +42,31 @@ onMount(() => {
   {#if !feedback && (!history || history.length === 0)}
     <div class="border rounded p-6 text-gray-500">No results yet. Upload submissions to see feedback here.</div>
   {/if}
-  <div class="flex justify-end mt-6">
-    <button class="bg-red-100 text-red-700 px-3 py-1 rounded hover:bg-red-200 transition" on:click={clearHistory} disabled={history.length === 0}>Clear History</button>
-  </div>
+  {#if history.length > 0}
+    <div class="flex justify-end mt-6">
+      <button class="bg-red-100 text-red-700 px-3 py-1 rounded hover:bg-red-200 transition" on:click={confirmClearHistory}>Clear History</button>
+    </div>
+  {/if}
   <AssessmentHistory {history} hideIfEmpty={true} on:delete={deleteEntry} />
 </section>
 
 <script lang="ts" context="module">
-export function clearHistory() {
-  localStorage.removeItem('assessmentHistory');
-  window.location.reload();
+export function confirmClearHistory() {
+  if (window.confirm('Are you sure you want to clear all assessment history? This action cannot be undone.')) {
+    localStorage.removeItem('assessmentHistory');
+    window.location.reload();
+  }
 }
 export function deleteEntry(event: CustomEvent) {
   const timestamp = event.detail;
-  let prev = [];
-  try {
-    prev = JSON.parse(localStorage.getItem('assessmentHistory') || '[]');
-  } catch {}
-
-  prev = prev.filter((entry: any) => entry.timestamp !== timestamp);
-  localStorage.setItem('assessmentHistory', JSON.stringify(prev));
-  window.location.reload();
+  if (window.confirm('Delete this assessment entry? This action cannot be undone.')) {
+    let prev = [];
+    try {
+      prev = JSON.parse(localStorage.getItem('assessmentHistory') || '[]');
+    } catch {}
+    prev = prev.filter((entry: any) => entry.timestamp !== timestamp);
+    localStorage.setItem('assessmentHistory', JSON.stringify(prev));
+    window.location.reload();
+  }
 }
 </script>