From 8f7551f4be4ed38c0ec6548f7c33d4fc5e574d8e Mon Sep 17 00:00:00 2001
From: Sean Morley <zipsm15@gmail.com>
Date: Thu, 26 Dec 2024 22:09:17 -0500
Subject: [PATCH] Enhance unlinked state logic for checklists and notes; update
 button layout in modals

---
 frontend/src/lib/components/ChecklistCard.svelte       | 2 +-
 frontend/src/lib/components/ChecklistModal.svelte      | 3 +--
 frontend/src/lib/components/NoteCard.svelte            | 2 +-
 frontend/src/lib/components/NoteModal.svelte           | 5 ++---
 frontend/src/lib/components/TransportationModal.svelte | 2 +-
 frontend/src/routes/collections/[id]/+page.svelte      | 2 +-
 6 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/frontend/src/lib/components/ChecklistCard.svelte b/frontend/src/lib/components/ChecklistCard.svelte
index 13814dc..37a024f 100644
--- a/frontend/src/lib/components/ChecklistCard.svelte
+++ b/frontend/src/lib/components/ChecklistCard.svelte
@@ -30,7 +30,7 @@
 				collection.start_date > checklist.date &&
 				collection.end_date > checklist.date;
 
-			unlinked = !!(startOutsideRange || endOutsideRange);
+			unlinked = !!(startOutsideRange || endOutsideRange || !checklist.date);
 		}
 	}
 
diff --git a/frontend/src/lib/components/ChecklistModal.svelte b/frontend/src/lib/components/ChecklistModal.svelte
index d6388dc..07cd468 100644
--- a/frontend/src/lib/components/ChecklistModal.svelte
+++ b/frontend/src/lib/components/ChecklistModal.svelte
@@ -293,10 +293,9 @@
 				{/if}
 
 				<div class="mt-4">
-					<button class="btn btn-neutral" on:click={close}>{$t('about.close')}</button>
 					<button class="btn btn-primary mr-1" disabled={isReadOnly} on:click={save}
 						>{$t('notes.save')}</button
-					>
+					><button class="btn btn-neutral" on:click={close}>{$t('about.close')}</button>
 				</div>
 			</form>
 		</div>
diff --git a/frontend/src/lib/components/NoteCard.svelte b/frontend/src/lib/components/NoteCard.svelte
index f513f78..9b6ba62 100644
--- a/frontend/src/lib/components/NoteCard.svelte
+++ b/frontend/src/lib/components/NoteCard.svelte
@@ -25,7 +25,7 @@
 			const endOutsideRange =
 				note.date && collection.start_date > note.date && collection.end_date > note.date;
 
-			unlinked = !!(startOutsideRange || endOutsideRange);
+			unlinked = !!(startOutsideRange || endOutsideRange || !note.date);
 		}
 	}
 
diff --git a/frontend/src/lib/components/NoteModal.svelte b/frontend/src/lib/components/NoteModal.svelte
index 8b37a7c..8344984 100644
--- a/frontend/src/lib/components/NoteModal.svelte
+++ b/frontend/src/lib/components/NoteModal.svelte
@@ -288,11 +288,10 @@
 
 				<!-- Action Buttons -->
 				<div class="mt-4">
-					<button class="btn btn-neutral" on:click={close}>
-						{$t('about.close')}
-					</button>
 					<button class="btn btn-primary mr-1" disabled={isReadOnly} on:click={save}>
 						{$t('notes.save')}
+					</button><button class="btn btn-neutral" on:click={close}>
+						{$t('about.close')}
 					</button>
 				</div>
 			</form>
diff --git a/frontend/src/lib/components/TransportationModal.svelte b/frontend/src/lib/components/TransportationModal.svelte
index a37c6bd..07291ea 100644
--- a/frontend/src/lib/components/TransportationModal.svelte
+++ b/frontend/src/lib/components/TransportationModal.svelte
@@ -589,7 +589,7 @@ it would also work to just use on:click on the MapLibre component itself. -->
 				<!-- Form Actions -->
 				<div class="mt-4">
 					<button type="submit" class="btn btn-primary">
-						{$t('adventures.save_next')}
+						{$t('notes.save')}
 					</button>
 					<button type="button" class="btn" on:click={close}>
 						{$t('about.close')}
diff --git a/frontend/src/routes/collections/[id]/+page.svelte b/frontend/src/routes/collections/[id]/+page.svelte
index 0265ba4..c8d6e2f 100644
--- a/frontend/src/routes/collections/[id]/+page.svelte
+++ b/frontend/src/routes/collections/[id]/+page.svelte
@@ -426,7 +426,7 @@
 	{/if}
 
 	{#if collection.description}
-		<div class="flex justify-center mt-4">
+		<div class="flex justify-center mt-4 max-w-screen-lg mx-auto">
 			<article
 				class="prose overflow-auto max-h-96 max-w-full p-4 border border-base-300 rounded-lg bg-base-300 mb-4"
 				style="overflow-y: auto;"