From eb63c4ef03d072d289c591371db2a0c04d8fab1c Mon Sep 17 00:00:00 2001
From: Gregor Vostrak <gregor@solidtime.io>
Date: Wed, 20 May 2026 15:24:09 +0200
Subject: [PATCH] fix light mode timesheet background and add missing
 aria-label

---
 resources/js/Components/Timesheet/TimesheetGrid.vue | 8 +++++---
 resources/js/Components/Timesheet/TimesheetRow.vue  | 1 +
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/resources/js/Components/Timesheet/TimesheetGrid.vue b/resources/js/Components/Timesheet/TimesheetGrid.vue
index c4774d06..cec3da5a 100644
--- a/resources/js/Components/Timesheet/TimesheetGrid.vue
+++ b/resources/js/Components/Timesheet/TimesheetGrid.vue
@@ -124,6 +124,7 @@ const emit = defineEmits<{
                         :create-client="createClient"
                         :organization-billable-rate="organization?.billable_rate ?? null"
                         :no-project-value="null"
+                        align="start"
                         @changed="(p, t) => emit('add-row', p, t)">
                         <template #trigger>
                             <Button variant="ghost" size="sm" class="text-text-secondary">
@@ -144,7 +145,7 @@ const emit = defineEmits<{
                     :key="dayIndex"
                     data-testid="timesheet_day_total"
                     :class="[
-                        'flex items-center justify-center border-t border-default-background-separator bg-secondary px-2 py-1 text-xs font-medium',
+                        'flex items-center justify-center border-t border-default-background-separator bg-background dark:bg-secondary px-2 py-1 text-xs font-medium',
                         weekDays[dayIndex] === todayDate
                             ? 'text-text-primary'
                             : 'text-text-secondary',
@@ -154,10 +155,11 @@ const emit = defineEmits<{
                     </span>
                 </div>
                 <div
-                    class="flex items-center justify-end border-t border-default-background-separator bg-secondary pl-3 pr-3 py-1 text-xs font-semibold text-text-primary">
+                    class="flex items-center justify-end border-t border-default-background-separator bg-background dark:bg-secondary pl-3 pr-3 py-1 text-xs font-semibold text-text-primary">
                     {{ weekTotalFormatted }}
                 </div>
-                <div class="border-t border-default-background-separator bg-secondary"></div>
+                <div
+                    class="border-t border-default-background-separator bg-background dark:bg-secondary"></div>
             </div>
         </div>
     </div>
diff --git a/resources/js/Components/Timesheet/TimesheetRow.vue b/resources/js/Components/Timesheet/TimesheetRow.vue
index f96b77e7..b3501e8a 100644
--- a/resources/js/Components/Timesheet/TimesheetRow.vue
+++ b/resources/js/Components/Timesheet/TimesheetRow.vue
@@ -124,6 +124,7 @@ function hasRunningEntry(dayIndex: number): boolean {
             <Button
                 variant="ghost"
                 size="icon"
+                aria-label="Remove row"
                 class="h-6 w-6 flex-shrink-0 opacity-0 group-hover:opacity-100 transition-opacity"
                 @click="emit('removeRow', row.key)">
                 <XMarkIcon class="h-3.5 w-3.5 text-icon-default" />