admin-mrrm/mrrmlabapp

Fork 0

[Bug] Einstellungs-Bereich: Settings-Navigation soll den Drawer/Sidebar kapern, statt eigener Top-/Side-Bar (Web) #348

New issue

Closed

opened 2026-05-19 22:44:38 +02:00 by admin-mrrm · 1 comment

admin-mrrm commented

2026-05-19 22:44:38 +02:00

Owner

Beobachtetes Verhalten

Im /einstellungen/*-Bereich erscheinen die Settings-Items (Mailkonten, Mail-Tags, Geschäfte) als horizontale Tab-Leiste oben in der Page. Der globale Drawer (Hamburger) zeigt unverändert die Top-Level-Items (Einkaufen, Todo, Listen, Mail, …). Auf Desktop existieren dadurch zwei Sidebars nebeneinander: globale Nav links + Settings-Nav 220 px daneben.

Erwartetes Verhalten

Wenn der User im Einstellungs-Bereich ist, kapert der Drawer/die Sidebar die Einstellungs-Items:

Auf Mobile: Hamburger → öffnet Drawer, Drawer zeigt Settings-Items (statt Top-Level-Items). Optional: oben im Drawer ein „← Zurück"-Eintrag, der zur Top-Level-Nav zurückkehrt.
Auf Desktop: linke Sidebar zeigt Settings-Items statt Top-Level-Items. Keine zweite Sidebar.
Keine separate Top-/Side-Bar in der Settings-Page — Einstellungs-Page nutzt die volle Content-Breite.

Das war die ursprüngliche Intention von #336 („damit Einstellungen nicht die Menü-Einträge bei der normalen Nutzung mischen") — aktuell sind sie aber gemischt sichtbar.

Reproduktion

App im Mobile-Viewport öffnen (app.mrrm.de)
Profil-Menü → „Einstellungen" → landet auf /einstellungen/mail
Settings-Items (Mailkonten / Mail-Tags / Geschäfte) erscheinen als horizontale Leiste oben
Hamburger antippen → Drawer zeigt weiter Einkaufen/Todo/Listen/Mail/Archiv/Pakete

Umgebung

App: web
Version / Commit: aktueller main (post-#336)
OS / Browser / Device: Android, Vivaldi auf app.mrrm.de

Code-Hinweis

apps/web/src/routes/einstellungen-layout.tsx:26-37 — eigene XStack-TopBar auf <$gtSm
apps/web/src/routes/einstellungen-layout.tsx:39-50 — eigene YStack-Sidebar auf $gtSm
apps/web/src/routes/layout.tsx — globaler Drawer/Sidebar weiß nichts vom aktiven Bereich

Lösungsskizze: Layout-Context (z.B. useMatches() aus TanStack Router) im globalen Layout auswerten und bei /einstellungen/* die Nav-Items austauschen. einstellungen-layout.tsx rendert dann nur noch <Outlet />.

Bezug

Einführung: #336 (App-Shell-Refactor)
Verwandt: #347 (Web-Drawer als Bottom-Sheet) — beide Drawer-Refactor sollten zusammen passieren
Parität: Mobile-App (#332 Todo-Drawer-Pattern) — Drawer-Kapern wäre dort äquivalent zu prüfen, falls Settings-Bereich auf Mobile dasselbe Symptom zeigt

## Beobachtetes Verhalten Im `/einstellungen/*`-Bereich erscheinen die Settings-Items (Mailkonten, Mail-Tags, Geschäfte) als **horizontale Tab-Leiste oben** in der Page. Der globale Drawer (Hamburger) zeigt unverändert die Top-Level-Items (Einkaufen, Todo, Listen, Mail, …). Auf Desktop existieren dadurch **zwei Sidebars nebeneinander**: globale Nav links + Settings-Nav 220 px daneben. ## Erwartetes Verhalten Wenn der User im Einstellungs-Bereich ist, **kapert** der Drawer/die Sidebar die Einstellungs-Items: - Auf Mobile: Hamburger → öffnet Drawer, Drawer zeigt **Settings-Items** (statt Top-Level-Items). Optional: oben im Drawer ein „← Zurück"-Eintrag, der zur Top-Level-Nav zurückkehrt. - Auf Desktop: linke Sidebar zeigt Settings-Items statt Top-Level-Items. Keine zweite Sidebar. - Keine separate Top-/Side-Bar in der Settings-Page — Einstellungs-Page nutzt die volle Content-Breite. Das war die ursprüngliche Intention von #336 („damit Einstellungen nicht die Menü-Einträge bei der normalen Nutzung mischen") — aktuell sind sie aber gemischt sichtbar. ## Reproduktion 1. App im Mobile-Viewport öffnen (`app.mrrm.de`) 2. Profil-Menü → „Einstellungen" → landet auf `/einstellungen/mail` 3. Settings-Items (Mailkonten / Mail-Tags / Geschäfte) erscheinen als horizontale Leiste oben 4. Hamburger antippen → Drawer zeigt weiter Einkaufen/Todo/Listen/Mail/Archiv/Pakete ## Umgebung - App: **web** - Version / Commit: aktueller `main` (post-#336) - OS / Browser / Device: Android, Vivaldi auf `app.mrrm.de` ## Code-Hinweis - `apps/web/src/routes/einstellungen-layout.tsx:26-37` — eigene `XStack`-TopBar auf `<$gtSm` - `apps/web/src/routes/einstellungen-layout.tsx:39-50` — eigene `YStack`-Sidebar auf `$gtSm` - `apps/web/src/routes/layout.tsx` — globaler Drawer/Sidebar weiß nichts vom aktiven Bereich **Lösungsskizze:** Layout-Context (z.B. `useMatches()` aus TanStack Router) im globalen Layout auswerten und bei `/einstellungen/*` die Nav-Items austauschen. `einstellungen-layout.tsx` rendert dann nur noch `<Outlet />`. ## Bezug - Einführung: #336 (App-Shell-Refactor) - Verwandt: #347 (Web-Drawer als Bottom-Sheet) — beide Drawer-Refactor sollten zusammen passieren - Parität: Mobile-App (#332 Todo-Drawer-Pattern) — Drawer-Kapern wäre dort äquivalent zu prüfen, falls Settings-Bereich auf Mobile dasselbe Symptom zeigt

admin-mrrm added this to the v0.4.5 — App-Shell & Listen-UX milestone

2026-05-19 22:44:38 +02:00

admin-mrrm added the

type/bug

area/ui

area/web

labels

2026-05-19 22:44:38 +02:00

admin-mrrm referenced this issue

2026-05-19 22:45:14 +02:00

[Bug] Profil-Menü (Popover) schließt nicht nach Klick auf „Einstellungen" oder „Konto" #349

admin-mrrm referenced this issue from a commit

2026-05-20 07:23:33 +02:00

fix(#348): Settings-Nav kapert den globalen Drawer/Sidebar

admin-mrrm referenced this issue

2026-05-20 07:23:44 +02:00

fix(#348): Settings-Nav kapert den globalen Drawer/Sidebar #358

admin-mrrm referenced this issue from a commit

2026-05-20 07:41:22 +02:00

Merge pull request 'fix(#348): Settings-Nav kapert den globalen Drawer/Sidebar' (#358) from fix/348-settings-nav-hijack-v2 into main

admin-mrrm referenced this issue

2026-05-20 07:50:55 +02:00

feat(app-shell): Navigation in Sidebar links + Profil-Menü rechts oben + separater Einstellungsbereich (Web + Mobile) #336

pm-bot added a new dependency

2026-05-24 18:15:54 +02:00

#336 feat(app-shell): Navigation in Sidebar links + Profil-Menü rechts oben + separater Einstellungsbereich (Web + Mobile)

pm-bot referenced this issue

2026-05-24 18:15:55 +02:00

[Sprint-Queue] v0.4.5 — Reihenfolge für Devs #400

pm-bot commented

2026-05-24 18:31:07 +02:00

Collaborator

PM-Housekeeping: Fix wurde bereits am 2026-05-20 via PR #358 (Commit a1433f2) auf main gemerged und ist seitdem in Prod ausgerollt. Issue wurde nur nicht automatisch geschlossen weil der Commit-Title fix(#348): statt Fixes #348 verwendet.

Schließe als verifiziert-implementiert. Falls bei dir lokal das Verhalten noch nicht funktioniert: bitte App-Cache leeren und neu laden — wieder öffnen wenn der Bug reproduzierbar bleibt.

**PM-Housekeeping:** Fix wurde bereits am 2026-05-20 via PR #358 (Commit `a1433f2`) auf `main` gemerged und ist seitdem in Prod ausgerollt. Issue wurde nur nicht automatisch geschlossen weil der Commit-Title `fix(#348):` statt `Fixes #348` verwendet. Schließe als verifiziert-implementiert. Falls bei dir lokal das Verhalten noch nicht funktioniert: bitte App-Cache leeren und neu laden — wieder öffnen wenn der Bug reproduzierbar bleibt.

pm-bot closed this issue

2026-05-24 18:31:07 +02:00

pm-bot referenced this issue

2026-05-25 11:41:38 +02:00

[arch-question] CI-Guard für Close-Keyword-Convention in Merge-Commits #406

~~pm-bot referenced this issue 2026-05-27 22:43:32 +02:00~~

Portfolio-Status KW 2026-W22 #412

pm-bot referenced this issue

2026-06-05 06:54:08 +02:00

feat(ci): pr-convention-check Drone-Step + Branch-Protection #434