From 9895e265144a1bfec967d12570ca875e2b3377c1 Mon Sep 17 00:00:00 2001
From: Liam Bigelow <40188355+bglw@users.noreply.github.com>
Date: Fri, 1 Jul 2022 16:57:15 +1200
Subject: [PATCH] Refining Pagefind UI

---
 pagefind_ui/svelte/filters.svelte | 90 ++++++++++++++++---------------
 pagefind_ui/svelte/ui.svelte      | 12 +++--
 2 files changed, 57 insertions(+), 45 deletions(-)

diff --git a/pagefind_ui/svelte/filters.svelte b/pagefind_ui/svelte/filters.svelte
index c5c5f578..0a360710 100644
--- a/pagefind_ui/svelte/filters.svelte
+++ b/pagefind_ui/svelte/filters.svelte
@@ -3,50 +3,53 @@
     export const selected_filters = {};
 </script>
 
-<fieldset class="pagefind-ui__filter-panel">
-    <legend class="pagefind-ui__filter-panel-label">Filters</legend>
-    {#if available_filters}
-        {#each Object.entries(available_filters) as [filter, values]}
-            <details class="pagefind-ui__filter-block">
-                <summary class="pagefind-ui__filter-name"
-                    >{filter.replace(/^(\w)/, (c) =>
-                        c.toLocaleUpperCase()
-                    )}</summary
-                >
-                <fieldset class="pagefind-ui__filter-group">
-                    <legend class="pagefind-ui__filter-group-label"
-                        >{filter}</legend
+{#if !available_filters || Object.entries(available_filters).length}
+    <fieldset class="pagefind-ui__filter-panel">
+        <legend class="pagefind-ui__filter-panel-label">Filters</legend>
+        {#if available_filters}
+            {#each Object.entries(available_filters) as [filter, values]}
+                <details class="pagefind-ui__filter-block">
+                    <summary class="pagefind-ui__filter-name"
+                        >{filter.replace(/^(\w)/, (c) =>
+                            c.toLocaleUpperCase()
+                        )}</summary
                     >
-                    {#each Object.entries(values) as [value, count]}
-                        <div
-                            class="pagefind-ui__filter-value"
-                            class:pagefind-ui__filter-value--checked={selected_filters[
-                                `${filter}:${value}`
-                            ]}
+                    <fieldset class="pagefind-ui__filter-group">
+                        <legend class="pagefind-ui__filter-group-label"
+                            >{filter}</legend
                         >
-                            <input
-                                class="pagefind-ui__filter-checkbox"
-                                type="checkbox"
-                                id="{filter}-{value}"
-                                name={filter}
-                                bind:checked={selected_filters[
+                        {#each Object.entries(values) as [value, count]}
+                            <div
+                                class="pagefind-ui__filter-value"
+                                class:pagefind-ui__filter-value--checked={selected_filters[
                                     `${filter}:${value}`
                                 ]}
-                                {value}
-                            />
-                            <label
-                                class="pagefind-ui__filter-label"
-                                for="{filter}-{value}">{value} ({count})</label
                             >
-                        </div>
-                    {/each}
-                </fieldset>
-            </details>
-        {/each}
-    {:else}
-        <p class="pagefind-ui__loading">..........</p>
-    {/if}
-</fieldset>
+                                <input
+                                    class="pagefind-ui__filter-checkbox"
+                                    type="checkbox"
+                                    id="{filter}-{value}"
+                                    name={filter}
+                                    bind:checked={selected_filters[
+                                        `${filter}:${value}`
+                                    ]}
+                                    {value}
+                                />
+                                <label
+                                    class="pagefind-ui__filter-label"
+                                    for="{filter}-{value}"
+                                    >{value} ({count})</label
+                                >
+                            </div>
+                        {/each}
+                    </fieldset>
+                </details>
+            {/each}
+        {:else}
+            <p class="pagefind-ui__loading">..........</p>
+        {/if}
+    </fieldset>
+{/if}
 
 <style>
     legend {
@@ -55,10 +58,10 @@
     }
     .pagefind-ui__filter-panel {
         min-width: min(calc(260px * var(--pagefind-ui-scale)), 100%);
-        gap: calc(40px * var(--pagefind-ui-scale));
         flex: 1;
         display: flex;
         flex-direction: column;
+        margin-top: calc(20px * var(--pagefind-ui-scale));
     }
     .pagefind-ui__filter-group {
         border: 0;
@@ -67,6 +70,9 @@
     .pagefind-ui__filter-block {
         padding: 0;
         display: block;
+        border-bottom: solid calc(2px * var(--pagefind-ui-scale))
+            var(--pagefind-ui-border);
+        padding: calc(20px * var(--pagefind-ui-scale)) 0;
     }
     .pagefind-ui__filter-name {
         font-size: calc(16px * var(--pagefind-ui-scale));
@@ -76,6 +82,7 @@
         list-style: none;
         font-weight: 700;
         cursor: pointer;
+        height: calc(24px * var(--pagefind-ui-scale));
     }
     .pagefind-ui__filter-name::after {
         position: absolute;
@@ -96,8 +103,7 @@
         display: flex;
         flex-direction: column;
         gap: calc(20px * var(--pagefind-ui-scale));
-        padding: calc(30px * var(--pagefind-ui-scale)) 0
-            calc(20px * var(--pagefind-ui-scale));
+        padding-top: calc(30px * var(--pagefind-ui-scale));
     }
     .pagefind-ui__filter-value {
         position: relative;
diff --git a/pagefind_ui/svelte/ui.svelte b/pagefind_ui/svelte/ui.svelte
index 7d90cd2c..6f321870 100644
--- a/pagefind_ui/svelte/ui.svelte
+++ b/pagefind_ui/svelte/ui.svelte
@@ -149,7 +149,7 @@
 <style>
     :root {
         --pagefind-ui-scale: 1;
-        --pagefind-ui-primary: #034ad8;
+        --pagefind-ui-primary: #393939;
         --pagefind-ui-text: #393939;
         --pagefind-ui-background: #ffffff;
         --pagefind-ui-border: #eeeeee;
@@ -207,7 +207,6 @@
     }
     .pagefind-ui__drawer {
         gap: calc(60px * var(--pagefind-ui-scale));
-        margin-top: calc(40px * var(--pagefind-ui-scale));
         display: flex;
         flex-direction: row;
         flex-wrap: wrap;
@@ -215,13 +214,18 @@
     .pagefind-ui__results-area {
         min-width: min(calc(400px * var(--pagefind-ui-scale)), 100%);
         flex: 1000;
+        margin-top: calc(20px * var(--pagefind-ui-scale));
     }
     .pagefind-ui__results {
         padding: 0;
     }
     .pagefind-ui__message {
+        box-sizing: content-box;
         font-size: calc(16px * var(--pagefind-ui-scale));
-        margin-bottom: calc(40px * var(--pagefind-ui-scale));
+        height: calc(24px * var(--pagefind-ui-scale));
+        padding: calc(20px * var(--pagefind-ui-scale)) 0;
+        display: flex;
+        align-items: center;
         font-weight: 700;
         margin-top: 0;
     }
@@ -234,6 +238,8 @@
         font-size: calc(16px * var(--pagefind-ui-scale));
         color: var(----pagefind-ui-primary);
         background: var(--pagefind-ui-background);
+        width: 100%;
+        text-align: center;
         font-weight: 700;
         cursor: pointer;
     }