marcel/mealprep
1
0
Fork 0
Code Issues 22 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(planner): map backend role 'planner' to 'planer' and enlarge nav buttons to 40px touch targets

Browse Source 
- hooks.server.ts: replace type-cast with actual mapping so isPlanner works
- planner page: set min-h/min-w 40px on prev/next/heute week buttons

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit was merged in pull request #44.
This commit is contained in:
Marcel Raddatz
2026-04-09 09:51:32 +02:00
parent e5d96cd85a
commit 5b8d336d21
3 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
frontend/src/hooks.server.test.ts
View File

@@ -79,7 +79,7 @@ describe('auth guard (hooks.server.ts handle)', () => {
displayName: 'Max',
householdId: 'h1',
householdName: 'Familie Müller',
householdRole: 'planer',
householdRole: 'planner',
email: 'max@example.com',
systemRole: 'user'
}

2
frontend/src/hooks.server.ts
View File

@@ -39,7 +39,7 @@ export const handle: Handle = async ({ event, resolve }) => {
event.locals.benutzer = {
id: user.id!,
name: user.displayName!,
rolle: (user.householdRole as 'planer' | 'mitglied') ?? 'mitglied'
rolle: user.householdRole === 'planner' ? 'planer' : 'mitglied'
};
event.locals.haushalt = {
id: user.householdId ?? undefined,

10
frontend/src/routes/(app)/planner/+page.svelte
View File

@@ -149,7 +149,7 @@
type="button"
onclick={() => navigateWeek('prev')}
aria-label="Vorherige Woche"
class="rounded-[var(--radius-md)] border border-[var(--color-border)] p-1.5 text-[var(--color-text)] hover:bg-[var(--color-surface)]"
class="flex min-h-[40px] min-w-[40px] items-center justify-center rounded-[var(--radius-md)] border border-[var(--color-border)] text-[var(--color-text)] hover:bg-[var(--color-surface)]"
>
‹
</button>
@@ -157,7 +157,7 @@
type="button"
onclick={() => navigateWeek('next')}
aria-label="Nächste Woche"
class="rounded-[var(--radius-md)] border border-[var(--color-border)] p-1.5 text-[var(--color-text)] hover:bg-[var(--color-surface)]"
class="flex min-h-[40px] min-w-[40px] items-center justify-center rounded-[var(--radius-md)] border border-[var(--color-border)] text-[var(--color-text)] hover:bg-[var(--color-surface)]"
>
›
</button>
@@ -279,7 +279,7 @@
type="button"
onclick={() => navigateWeek('prev')}
aria-label="Vorherige Woche"
class="rounded-[var(--radius-md)] border border-[var(--color-border)] px-3 py-1.5 text-[13px] font-medium tracking-[0.04em] font-[var(--font-sans)] text-[var(--color-text)] hover:bg-[var(--color-surface)]"
class="flex min-h-[40px] min-w-[40px] items-center justify-center rounded-[var(--radius-md)] border border-[var(--color-border)] text-[13px] font-medium tracking-[0.04em] font-[var(--font-sans)] text-[var(--color-text)] hover:bg-[var(--color-surface)]"
>
‹
</button>
@@ -288,14 +288,14 @@
type="button"
onclick={() => navigateWeek('next')}
aria-label="Nächste Woche"
class="rounded-[var(--radius-md)] border border-[var(--color-border)] px-3 py-1.5 text-[13px] font-medium tracking-[0.04em] font-[var(--font-sans)] text-[var(--color-text)] hover:bg-[var(--color-surface)]"
class="flex min-h-[40px] min-w-[40px] items-center justify-center rounded-[var(--radius-md)] border border-[var(--color-border)] text-[13px] font-medium tracking-[0.04em] font-[var(--font-sans)] text-[var(--color-text)] hover:bg-[var(--color-surface)]"
>
›
</button>
<button
type="button"
onclick={() => navigateWeek('today')}
class="rounded-[var(--radius-md)] border border-[var(--color-border)] px-3 py-1.5 text-[13px] font-medium tracking-[0.04em] font-[var(--font-sans)] text-[var(--color-text)] hover:bg-[var(--color-surface)]"
class="flex min-h-[40px] items-center rounded-[var(--radius-md)] border border-[var(--color-border)] px-3 text-[13px] font-medium tracking-[0.04em] font-[var(--font-sans)] text-[var(--color-text)] hover:bg-[var(--color-surface)]"
>
Heute
</button>