From 08d07052c4df071e2efdefdb3fa23fcad3ee34e7 Mon Sep 17 00:00:00 2001
From: Sander Eikemo Andreassen <seandreassen04@gmail.com>
Date: Thu, 7 Nov 2024 17:42:02 +0100
Subject: [PATCH] refactor: move aria label and remove unnecessary span

---
 src/components/shift-schedule/AdministratorMenu.tsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/components/shift-schedule/AdministratorMenu.tsx b/src/components/shift-schedule/AdministratorMenu.tsx
index 6b05a89..3548912 100644
--- a/src/components/shift-schedule/AdministratorMenu.tsx
+++ b/src/components/shift-schedule/AdministratorMenu.tsx
@@ -30,11 +30,11 @@ function AdministratorMenu({ t }: AdministratorMenuProps) {
       <div className='mx-1 flex justify-between'>
         <span className='my-auto font-semibold text-xl'>{t.label}</span>
         <CollapsibleTrigger asChild>
-          <Button variant='ghost'>
+          <Button variant='ghost' aria-label={isOpen ? t.close : t.open}>
             {isOpen ? (
-              <ChevronUpIcon aria-label={t.close} className='size-4' />
+              <ChevronUpIcon aria-hidden='true' className='size-4' />
             ) : (
-              <ChevronDownIcon aria-label={t.open} className='size-4' />
+              <ChevronDownIcon aria-hidden='true' className='size-4' />
             )}
           </Button>
         </CollapsibleTrigger>
@@ -42,7 +42,7 @@ function AdministratorMenu({ t }: AdministratorMenuProps) {
       <CollapsibleContent className='mt-2'>
         <Button variant='link' className='flex gap-3 text-destructive'>
           <Trash2Icon />
-          <span>{t.clearShiftSchedule}</span>
+          {t.clearShiftSchedule}
         </Button>
       </CollapsibleContent>
     </Collapsible>