Handle empty queries better

MuckRock · Apr 24, 2024 · 328a527 · 328a527
1 parent e84ce97
commit 328a527
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/src/routes/app/add-ons/+page.svelte b/src/routes/app/add-ons/+page.svelte
@@ -39,7 +39,6 @@
     const url = new URL($page.url); // make a copy
     const formData = new FormData(event.currentTarget as HTMLFormElement);
     const query = formData.get("query") ?? "";
-    if (!query) return;
     url.searchParams.set("query", query as string);
     goto(url);
   }

diff --git a/src/routes/app/projects/+page.svelte b/src/routes/app/projects/+page.svelte
@@ -22,7 +22,6 @@
     const url = new URL($page.url); // make a copy
     const formData = new FormData(event.currentTarget as HTMLFormElement);
     const query = formData.get("query") ?? "";
-    if (!query) return;
     url.searchParams.set("query", query as string);
     goto(url);
   }

diff --git a/src/routes/app/projects/[id]-[slug]/+page.ts b/src/routes/app/projects/[id]-[slug]/+page.ts
@@ -8,8 +8,8 @@ export async function load({ params, url, parent, fetch }) {
   const breadcrumbs = await breadcrumbTrail(parent, [
     { href: url.pathname, title: project.title },
   ]);
-  const query = url.searchParams.get("q");
-  const cursor = url.searchParams.get("cursor");
+  const query = url.searchParams.get("q") ?? "";
+  const cursor = url.searchParams.get("cursor") ?? "";
   const documents = search(query, { cursor, project: project.id });
   return {
     breadcrumbs,