From 131cd2f4a27b6e6bde3524f960b1547d1ab565ff Mon Sep 17 00:00:00 2001
From: Sander Eikemo Andreassen <seandreassen04@gmail.com>
Date: Wed, 30 Oct 2024 16:04:38 +0100
Subject: [PATCH] fix: add label to administrator menu button and fix
 formatting

---
 src/app/[locale]/(default)/shift-schedule/page.tsx  |  4 ++--
 src/components/shift-schedule/AdministratorMenu.tsx | 13 +++++++++----
 2 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/src/app/[locale]/(default)/shift-schedule/page.tsx b/src/app/[locale]/(default)/shift-schedule/page.tsx
index 24d8de4..06c79f6 100644
--- a/src/app/[locale]/(default)/shift-schedule/page.tsx
+++ b/src/app/[locale]/(default)/shift-schedule/page.tsx
@@ -29,9 +29,9 @@ export default async function ShiftSchedulePage({
   return (
     <>
       <AdministratorMenu
-        messages = {{
+        messages={{
           administratorMenu: t('administratorMenu.administratorMenu'),
-          clearShiftSchedule: t('administratorMenu.clearShiftSchedule')
+          clearShiftSchedule: t('administratorMenu.clearShiftSchedule'),
         }}
       />
       <ScheduleTable week={shiftScheduleMockData} />
diff --git a/src/components/shift-schedule/AdministratorMenu.tsx b/src/components/shift-schedule/AdministratorMenu.tsx
index 83fb98e..1445717 100644
--- a/src/components/shift-schedule/AdministratorMenu.tsx
+++ b/src/components/shift-schedule/AdministratorMenu.tsx
@@ -6,7 +6,6 @@ import {
   CollapsibleContent,
   CollapsibleTrigger,
 } from '@/components/ui/Collapsible';
-import { Separator } from '@/components/ui/Separator';
 import { ChevronDownIcon, ChevronUpIcon, Trash2Icon } from 'lucide-react';
 import { useState } from 'react';
 
@@ -24,7 +23,7 @@ function AdministratorMenu({ messages }: AdministratorMenuProps) {
     <Collapsible
       open={isOpen}
       onOpenChange={setIsOpen}
-      className='data-[state] mx-auto xs:mx-8 my-8 rounded border p-3'
+      className='mx-auto xs:mx-8 my-8 rounded border p-3'
     >
       <section className='mx-1 flex justify-between'>
         <span className='my-auto font-semibold text-xl'>
@@ -33,9 +32,15 @@ function AdministratorMenu({ messages }: AdministratorMenuProps) {
         <CollapsibleTrigger asChild>
           <Button variant='ghost'>
             {isOpen ? (
-              <ChevronUpIcon className='size-4' />
+              <ChevronUpIcon
+                aria-label='Close Administrator Menu'
+                className='size-4'
+              />
             ) : (
-              <ChevronDownIcon className='size-4' />
+              <ChevronDownIcon
+                aria-label='Open Administrator Menu'
+                className='size-4'
+              />
             )}
           </Button>
         </CollapsibleTrigger>