From e65883cc9ee758951efce86fae8dcf6dae81eaff Mon Sep 17 00:00:00 2001
From: Admin <admin@kalekber.cc>
Date: Thu, 16 Apr 2026 12:34:55 +0500
Subject: [PATCH] feat(catalogue): UX improvements and bug fixes

- Persist audio filter in URL (?audio=1) via history.replaceState
- Show total novel count in browse mode subtitle
- Close filter panel automatically on Apply
- Replace missing-cover SVG placeholder with styled first-letter avatar
- Add forbidden scrape badge to list view (was missing, grid had it)
- Carry audio param through applyFilters() navigation

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 ui/src/routes/catalogue/+page.server.ts |  4 ++-
 ui/src/routes/catalogue/+page.svelte    | 37 +++++++++++++++----------
 2 files changed, 26 insertions(+), 15 deletions(-)

diff --git a/ui/src/routes/catalogue/+page.server.ts b/ui/src/routes/catalogue/+page.server.ts
index a727892..90c3afd 100644
--- a/ui/src/routes/catalogue/+page.server.ts
+++ b/ui/src/routes/catalogue/+page.server.ts
@@ -16,6 +16,7 @@ export const load: PageServerLoad = async ({ url, locals }) => {
 	const sort = url.searchParams.get('sort') ?? 'popular';
 	const status = url.searchParams.get('status') ?? 'all';
 	const q = url.searchParams.get('q') ?? '';
+	const audioOnly = url.searchParams.get('audio') === '1';
 
 	const params = new URLSearchParams({ page, genre, sort, status });
 	if (q.trim().length >= 2) {
@@ -64,7 +65,8 @@ export const load: PageServerLoad = async ({ url, locals }) => {
 		isAdmin: locals.user?.role === 'admin',
 		searchQuery: q.trim().length >= 2 ? q.trim() : '',
 		searchLocalCount: 0,
-		searchRemoteCount: 0
+		searchRemoteCount: 0,
+		audioOnly
 	};
 };
 
diff --git a/ui/src/routes/catalogue/+page.svelte b/ui/src/routes/catalogue/+page.svelte
index 4533cb0..f4affb1 100644
--- a/ui/src/routes/catalogue/+page.svelte
+++ b/ui/src/routes/catalogue/+page.svelte
@@ -21,11 +21,13 @@
 	});
 
 	function applyFilters() {
+		filtersOpen = false;
 		const params = new URLSearchParams();
 		params.set('sort', filterSort);
 		params.set('genre', filterGenre);
 		params.set('status', filterStatus);
 		params.set('page', '1');
+		if (filterAudioOnly) params.set('audio', '1');
 		goto(`/catalogue?${params.toString()}`);
 	}
 
@@ -215,7 +217,7 @@
 
 	// ── Audio-available set ───────────────────────────────────────────────────
 	let audioSlugs = $state<Set<string>>(new Set());
-	let filterAudioOnly = $state(false);
+	let filterAudioOnly = $state(untrack(() => data.audioOnly));
 
 	$effect(() => {
 		fetch('/api/audio/slugs')
@@ -224,6 +226,17 @@
 			.catch(() => { /* non-critical */ });
 	});
 
+	function toggleAudio() {
+		filterAudioOnly = !filterAudioOnly;
+		const u = new URL(window.location.href);
+		if (filterAudioOnly) {
+			u.searchParams.set('audio', '1');
+		} else {
+			u.searchParams.delete('audio');
+		}
+		history.replaceState({}, '', u.toString());
+	}
+
 	const displayedNovels = $derived(
 		filterAudioOnly ? novels.filter((n) => audioSlugs.has(n.slug)) : novels
 	);
@@ -249,7 +262,7 @@
 				{m.catalogue_rank_no_data_body()}
 			{/if}
 		{:else}
-			{m.catalogue_browse_source()}
+			{m.catalogue_browse_source()}{#if data.total > 0}&nbsp;<span class="text-(--color-muted) text-xs">{data.total.toLocaleString()} novels</span>{/if}
 		{/if}
 	</p>
 </div>
@@ -349,7 +362,7 @@
 	<!-- Audio-only filter toggle -->
 	{#if audioSlugs.size > 0}
 		<button
-			onclick={() => (filterAudioOnly = !filterAudioOnly)}
+			onclick={toggleAudio}
 			title="Show only books with audio"
 			class="flex items-center gap-1.5 px-2.5 py-2 rounded border text-sm transition-colors shrink-0
 				{filterAudioOnly
@@ -503,7 +516,7 @@
 					{m.catalogue_rank_run_scrape_user()}
 				{/if}
 			{:else if filterAudioOnly}
-				<button onclick={() => (filterAudioOnly = false)} class="text-(--color-brand) hover:underline">Clear audio filter</button>
+				<button onclick={toggleAudio} class="text-(--color-brand) hover:underline">Clear audio filter</button>
 			{:else}
 				{m.catalogue_no_results_filters()}
 			{/if}
@@ -531,11 +544,8 @@
 							loading="lazy"
 						/>
 					{:else}
-						<div class="w-full h-full flex items-center justify-center text-(--color-muted)">
-							<svg class="w-12 h-12" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-								<path stroke-linecap="round" stroke-linejoin="round" stroke-width="1.5"
-									d="M12 6.253v13m0-13C10.832 5.477 9.246 5 7.5 5S4.168 5.477 3 6.253v13C4.168 18.477 5.754 18 7.5 18s3.332.477 4.5 1.253m0-13C13.168 5.477 14.754 5 16.5 5c1.747 0 3.332.477 4.5 1.253v13C19.832 18.477 18.247 18 16.5 18c-1.746 0-3.332.477-4.5 1.253" />
-							</svg>
+						<div class="w-full h-full flex items-center justify-center bg-(--color-surface-3)">
+							<span class="text-5xl font-bold text-(--color-muted) select-none opacity-50">{novel.title.charAt(0).toUpperCase()}</span>
 						</div>
 					{/if}
 					{#if novel.rank}
@@ -622,11 +632,8 @@
 					{#if novel.cover}
 						<img src={novel.cover} alt={novel.title} class="w-full h-full object-cover" loading="lazy" />
 					{:else}
-						<div class="w-full h-full flex items-center justify-center text-(--color-muted)">
-							<svg class="w-5 h-5" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-								<path stroke-linecap="round" stroke-linejoin="round" stroke-width="1.5"
-									d="M12 6.253v13m0-13C10.832 5.477 9.246 5 7.5 5S4.168 5.477 3 6.253v13C4.168 18.477 5.754 18 7.5 18s3.332.477 4.5 1.253m0-13C13.168 5.477 14.754 5 16.5 5c1.747 0 3.332.477 4.5 1.253v13C19.832 18.477 18.247 18 16.5 18c-1.746 0-3.332.477-4.5 1.253" />
-							</svg>
+						<div class="w-full h-full flex items-center justify-center bg-(--color-surface-3)">
+							<span class="text-xl font-bold text-(--color-muted) select-none opacity-50">{novel.title.charAt(0).toUpperCase()}</span>
 						</div>
 					{/if}
 					{#if isLoading}
@@ -688,6 +695,8 @@
 							<span class="text-xs text-emerald-400 font-medium">{m.catalogue_scrape_queued_badge()}</span>
 						{:else if scrapeResult[novel.slug] === 'busy'}
 							<span class="text-xs text-yellow-400 font-medium">{m.catalogue_scrape_busy_list()}</span>
+						{:else if scrapeResult[novel.slug] === 'forbidden'}
+							<span class="text-xs text-(--color-danger) font-medium">{m.catalogue_scrape_forbidden_badge()}</span>
 						{:else if scrapeResult[novel.slug] === 'error'}
 							<span class="text-xs text-(--color-danger) font-medium">{m.common_error()}</span>
 						{:else}