From 61da21b7db60244be174287eb39b4a368ca30fc6 Mon Sep 17 00:00:00 2001
From: TadiosT <tadios.temesgen@gmail.com>
Date: Tue, 10 Jun 2025 05:45:29 +0100
Subject: [PATCH] fix: Style and type check changes.

Co-Authored By: Caspar Asaam <caspar@dyn3159-95.wlan.ic.ac.uk>
---
 src/lib/components/inputs/TagFilter.svelte | 48 +------------------
 src/lib/index.ts                           |  2 +-
 src/routes/+page.svelte                    | 55 +++++++++++-----------
 3 files changed, 30 insertions(+), 75 deletions(-)

diff --git a/src/lib/components/inputs/TagFilter.svelte b/src/lib/components/inputs/TagFilter.svelte
index 5d82fc8..4648fd1 100644
--- a/src/lib/components/inputs/TagFilter.svelte
+++ b/src/lib/components/inputs/TagFilter.svelte
@@ -1,6 +1,6 @@
 <script lang="ts">
     import crossUrl from "$lib/assets/cross.svg";
-    
+
     export let selectedTags: string[] = [];
     export let tagFilter: string;
     export let tagFilterElem;
@@ -8,12 +8,10 @@
     export let dropdownVisible: boolean;
     export let deleteTag;
     export let addTag;
-    
-  
 </script>
 
 <form>
-  <div class="tagDisplay">
+    <div class="tagDisplay">
         {#each selectedTags as tagName (tagName)}
             <button class="tag" onclick={deleteTag(tagName)} type="button">
                 {tagName}
@@ -59,7 +57,6 @@
             </div>
         {/if}
     </div>
-  
 </form>
 
 <style>
@@ -72,11 +69,6 @@
         margin: 0 auto;
     }
 
-    label {
-        color: #ffffff;
-        margin-top: 0.5rem;
-    }
-    
     .tagDisplay {
         display: flex;
         gap: 0.4rem;
@@ -155,40 +147,4 @@
     .avaliableTag:last-child {
         padding-bottom: 0.6rem;
     }
-
-    .compulsoryTags {
-        display: grid;
-        gap: 0.4rem;
-        border-radius: 0.5rem;
-        background-color: none;
-        width: 100%;
-        font-size: 1rem;
-        grid-template-columns: repeat(3, 1fr);
-    }
-
-    .compulsoryContainer {
-        display: flex;
-        flex-direction: column;
-        align-items: left;
-        justify-content: top;
-        border-radius: 0.5rem;
-        background-color: none;
-    }
-
-    .compulsoryTagSelect {
-        width: 100%;
-        height: 100%;
-        padding: 0.5rem;
-        border-radius: 0.5rem;
-        border: 2px solid #eaffeb;
-        background: none;
-        color: #eaffeb;
-        font-size: 0.9rem;
-        text-align: left;
-    }
-
-    .compulsoryTagSelect option {
-        background-color: #2e4653;
-        color: #eaffeb;
-    }
 </style>
diff --git a/src/lib/index.ts b/src/lib/index.ts
index f209ffe..3264a9e 100644
--- a/src/lib/index.ts
+++ b/src/lib/index.ts
@@ -25,4 +25,4 @@ export const volumeTags = ["Silent", "Quiet", "Some Noise", "Loud"];
 export const wifiTags = ["Good WiFi", "Moderate WiFi", "Bad WiFi", "No WiFi"];
 export const powerOutletTags = ["Many Outlets", "Some Outlets", "No Outlets"];
 
-export const allTags = [...availableStudySpaceTags, ...volumeTags, ...wifiTags, ...powerOutletTags]
+export const allTags = [...availableStudySpaceTags, ...volumeTags, ...wifiTags, ...powerOutletTags];
diff --git a/src/routes/+page.svelte b/src/routes/+page.svelte
index 84356a1..3160ef7 100644
--- a/src/routes/+page.svelte
+++ b/src/routes/+page.svelte
@@ -4,37 +4,36 @@
     import crossUrl from "$lib/assets/cross.svg";
     import Navbar from "$lib/components/Navbar.svelte";
     import TagFilter from "$lib/components/inputs/TagFilter.svelte";
-    import {availableStudySpaceTags} from "$lib"
-    import {allTags} from "$lib"
+    import { allTags } from "$lib";
 
     const { data } = $props();
-    const { studySpaces, supabase } = $derived(data);  
-    
-    let selectedTags = $state<string[]>([])
+    const { studySpaces, supabase } = $derived(data);
+
+    let selectedTags = $state<string[]>([]);
     let tagFilter = $state("");
-    let tagFilterElem = $state<HTMLInputElement>()
+    let tagFilterElem = $state<HTMLInputElement>();
     let filteredTags = $derived(
         allTags
             .filter((tag) => tag.toLowerCase().includes(tagFilter.toLowerCase()))
             .filter((tag) => allTags.includes(tag))
-        )
+    );
 
     let filteredStudySpaces = $derived(() =>
         selectedTags.length === 0
             ? studySpaces
-            : studySpaces.filter(space => {
-                const allTags = [
-                ...(space.tags || []),
-                space.volume,
-                space.wifi,
-                space.power
-                ].filter(Boolean); 
+            : studySpaces.filter((space) => {
+                  const allTags = [
+                      ...(space.tags || []),
+                      space.volume,
+                      space.wifi,
+                      space.power
+                  ].filter(Boolean);
 
-                return selectedTags.every(tag => allTags.includes(tag));
-            })
+                  return selectedTags.every((tag) => allTags.includes(tag));
+              })
     );
 
-    let dropdownVisible = $state(false)
+    let dropdownVisible = $state(false);
 
     function deleteTag(tagName: string) {
         return () => {
@@ -52,24 +51,24 @@
     }
 </script>
 
-<Navbar>   
+<Navbar>
     <a href="/space">
         <img src={crossUrl} alt="new" class="new-space" />
-    </a>   
+    </a>
 </Navbar>
 
 <main>
-    <TagFilter 
-        selectedTags={selectedTags}
-        tagFilter={tagFilter}
-        tagFilterElem={tagFilterElem}
-        filteredTags={filteredTags}
-        dropdownVisible={dropdownVisible}
-        deleteTag={deleteTag}
-        addTag={addTag}
+    <TagFilter
+        {selectedTags}
+        {tagFilter}
+        {tagFilterElem}
+        {filteredTags}
+        {dropdownVisible}
+        {deleteTag}
+        {addTag}
     />
 
-   {#each filteredStudySpaces() as studySpace (studySpace.id)}
+    {#each filteredStudySpaces() as studySpace (studySpace.id)}
         {@const imgUrl =
             studySpace.study_space_images.length > 0
                 ? supabase.storage