From f3285c21fd2fb930714eb128345aeffb5c0733b7 Mon Sep 17 00:00:00 2001
From: Antoine Phan <antoine.phan@mail.mcgill.ca>
Date: Thu, 12 Jun 2025 15:51:11 -0400
Subject: [PATCH] Update the styling on FAQ Accordion

---
 src/app.css                        |   2 +-
 src/components/FAQAccordion.svelte | 138 ++++++++++-------------------
 src/routes/+page.svelte            |  40 ++++-----
 3 files changed, 64 insertions(+), 116 deletions(-)

diff --git a/src/app.css b/src/app.css
index af4cd33..936a9b8 100644
--- a/src/app.css
+++ b/src/app.css
@@ -8,8 +8,8 @@
     --color-ecsess-400: #5C8A5C;
     --color-ecsess-600: #3B6A3A;
     --color-ecsess-800: #0A3D2A;
-    --color-ecsess-black-hover:#2c2c2c;
     --color-ecsess-black: #1F1F1F;
+    --color-ecsess-black-hover:#2c2c2c;
 }
 
 * {
diff --git a/src/components/FAQAccordion.svelte b/src/components/FAQAccordion.svelte
index c56bbba..a4d5669 100644
--- a/src/components/FAQAccordion.svelte
+++ b/src/components/FAQAccordion.svelte
@@ -1,101 +1,55 @@
 <script>
-  import { slide } from "svelte/transition";
-  import '/src/app.css';
-  export let entry;
-  const { title, description } = entry;
-  let isOpen = false;
-  const toggle = () => (isOpen = !isOpen);
+	import { slide } from 'svelte/transition';
+	let isOpen = false;
+	export let entry;
+	const { title, description } = entry;
+	const toggle = () => (isOpen = !isOpen);
 </script>
 
-<div class="accordion-wrapper">
-  <div class="accordion-item {isOpen ? 'open' : ''}">
-    <button
-      class="accordion-header"
-      on:click={toggle}
-      aria-expanded={isOpen}
-    >
-      <span class="title">{title}</span>
-      <svg
-        class="accordion-icon"
-        width="20"
-        height="20"
-        fill="none"
-        stroke-linecap="round"
-        stroke-linejoin="round"
-        stroke-width="2"
-        viewBox="0 0 24 24"
-        stroke="currentColor"
-      >
-        <path d="M9 5l7 7-7 7" />
-      </svg>
-    </button>
+<div class="accordion-wrapper w-[500px]">
+	<div
+		class="accordion-item
+    border-ecsess-200 mb-4 overflow-hidden rounded-xl border-2
+    {isOpen ? '' : 'open'}"
+	>
+		<button
+			class="accordion-header hover:bg-ecsess-black-hover
+            flex h-16 w-full cursor-pointer items-center justify-between
+            px-4 transition-colors ease-in-out"
+			on:click={toggle}
+			aria-expanded={isOpen}
+		>
+			<span class="font-bold text-xl">{title}</span>
+			<svg
+				class="accordion-icon transform-cpu ease-in-out transition-all"
+				width="20"
+				height="20"
+				fill="none"
+				stroke-linecap="round"
+				stroke-linejoin="round"
+				stroke-width="2"
+				viewBox="0 0 24 24"
+				stroke="currentColor"
+			>
+				<path d="M9 5l7 7-7 7" />
+			</svg>
+		</button>
 
-    {#if isOpen}
-      <div class="accordion-content" transition:slide={{ duration: 300 }}>
-        <p>{@html description}</p>
-      </div>
-    {/if}
-  </div>
+		{#if isOpen}
+			<div class="accordion-content max-h-fit border-t-2 border-t-ecsess-200 p-4 bg-transparent transition-all ease-in" transition:slide={{ duration: 300 }}>
+				<p>{@html description}</p>
+			</div>
+		{/if}
+	</div>
 </div>
 
 <style>
-  .accordion-wrapper {
-    width: 500px;
-  }
+	[aria-expanded='true'] .accordion-icon {
+		transform: rotate(90deg);
+	}
 
-  .accordion-item {
-    border: 2px solid var(--color-ecsess-200);
-    border-radius: 0.75rem;
-    overflow: hidden;
-    margin-bottom: 1rem;
-  }
-
-  .accordion-header {
-    display: flex;
-    justify-content: space-between;
-    align-items: center;
-    height: 64px;
-    padding: 0 1rem;
-    width: 100%;
-    color: inherit;
-    cursor: pointer;
-    background-color: transparent;
-    transition: background-color 0.3s ease;
-  }
-
-  .accordion-header:hover {
-   background-color: var(--color-ecsess-black-hover);
-  }
-
-  .accordion-icon {
-    transition: transform 0.3s ease;
-  }
-
-  [aria-expanded="true"] .accordion-icon {
-    transform: rotate(90deg);
-  }
-
-  .accordion-content {
-    max-height: 100px;
-    overflow-y: auto;
-    padding: 1rem;
-    border-top: 1px solid var(--color-ecsess-200);
-    color: var(--color-ecsess-200);
-    background-color: transparent;
-    transition: background-color 0.3s ease;
-  }
-
-  .accordion-item.open .accordion-header,
-  .accordion-item.open .accordion-content {
-    background-color: var(--color-ecsess-black-hover);
-  }
-
-  .title {
-    font-weight: bold;
-    font-size: 1.1rem;
-  }
-
-  p {
-    margin: 0;
-  }
+	.accordion-item.open .accordion-header,
+	.accordion-item.open .accordion-content {
+		background-color: var(--color-ecsess-black-hover);
+	}
 </style>
diff --git a/src/routes/+page.svelte b/src/routes/+page.svelte
index 8ad857f..eb0d88d 100644
--- a/src/routes/+page.svelte
+++ b/src/routes/+page.svelte
@@ -7,28 +7,23 @@
 	let { data } = $props();
 
 	// Getting info from ECSESS CMS
-	let longAnswer = 'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.';
-	const query = '_type:homepage';
-	
-	const faqs = [
-    {
-      q: "A fequently asked question?",
-      a: "This is a short answer",
-    },
-    {
-      q: "A fequently asked question?",
-      a: "This is a medium length answer. Lor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.",
-    }, 
-	{
-      q: "A fequently asked question?",
-      a: "This is a long answer. "+longAnswer
-    }
-  ];
+	let longAnswer =
+		'Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.';
 
-	let description = $state();
-	description = fetch(
-		`https://${import.meta.env.SANITY_ID}.api.sanity.io/vX/data/query/production?query=${query}&perspective=drafts`
-	).then((res) => res.json).then((json) => json.toString());
+	const faqs = [
+		{
+			q: 'A fequently asked question?',
+			a: 'This is a short answer'
+		},
+		{
+			q: 'A fequently asked question?',
+			a: 'This is a medium length answer. Lor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.'
+		},
+		{
+			q: 'A fequently asked question?',
+			a: 'This is a long answer. ' + longAnswer
+		}
+	];
 </script>
 
 <title> McGill ECSESS </title>
@@ -52,7 +47,7 @@
 	<div>
 		<h1>FAQ</h1>
 		{#each faqs as { q, a }}
-		<FaqAccordion entry={{ title: q, description: a }} />
+			<FaqAccordion entry={{ title: q, description: a }} />
 		{/each}
 	</div>
 </Section>
@@ -63,4 +58,3 @@
 		<p>Under development</p>
 	</div>
 </Section>
-