admin-mrrm/mrrmlabapp

Fork 0

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

New issue

Closed

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

admin-mrrm commented

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

Owner

Beobachtetes Verhalten

Nach Klick auf „Einstellungen" oder „Konto" im Profil-Popover navigiert die App korrekt zur Ziel-Route — das Popover bleibt aber offen und überlagert die Page (siehe Screenshot zu #348).

Erwartetes Verhalten

Popover schließt sich nach jeder Item-Auswahl (Standardverhalten für Dropdown-Menus). Nur „Abmelden" funktioniert korrekt, da es ein Button ist (kein Link).

Reproduktion

Profil-Avatar (MM) oben rechts antippen
„Einstellungen" antippen → landet auf /einstellungen/mail
Popover bleibt sichtbar geöffnet, bis manuell außerhalb getippt wird

Umgebung

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

Code-Hinweis

apps/web/src/components/profile-menu.tsx:55-64:

<Popover.Close asChild>
  <Link to="/einstellungen/mail" style={...}>
    <MenuItem label="Einstellungen" />
  </Link>
</Popover.Close>

Das asChild-Pattern erwartet, dass das Child Event-Handler (insb. onPress/onClick) korrekt mergt. TanStack-Router <Link> rendert ein nativen <a> und ruft beim Klick preventDefault + eigenen Navigations-Handler auf — der Popover-Close-Handler von Tamagui wird dabei vermutlich nicht ausgelöst.

Lösungsskizze (Optionen):

Popover über open/onOpenChange-State steuern und setOpen(false) im Link-onClick aufrufen
Link in einen Button wrappen, der router.navigate() aufruft (dann funktioniert Popover.Close asChild mit Button-Trigger)
Popover.Close als äußeres Wrapper-Element verwenden (z.B. um den MenuItem-YStack statt um den Link)

Bezug

Einführung: #336 (App-Shell-Refactor — Profil-Menü neu)
Begleit-Bug zu #348 (Settings-Layout)

## Beobachtetes Verhalten Nach Klick auf „Einstellungen" oder „Konto" im Profil-Popover navigiert die App korrekt zur Ziel-Route — das **Popover bleibt aber offen** und überlagert die Page (siehe Screenshot zu #348). ## Erwartetes Verhalten Popover schließt sich nach jeder Item-Auswahl (Standardverhalten für Dropdown-Menus). Nur „Abmelden" funktioniert korrekt, da es ein `Button` ist (kein `Link`). ## Reproduktion 1. Profil-Avatar (MM) oben rechts antippen 2. „Einstellungen" antippen → landet auf `/einstellungen/mail` 3. Popover bleibt sichtbar geöffnet, bis manuell außerhalb getippt wird ## Umgebung - App: **web** - Version / Commit: aktueller `main` (post-#336) - OS / Browser / Device: Android, Vivaldi auf `app.mrrm.de` ## Code-Hinweis `apps/web/src/components/profile-menu.tsx:55-64`: ```tsx <Popover.Close asChild> <Link to="/einstellungen/mail" style={...}> <MenuItem label="Einstellungen" /> </Link> </Popover.Close> ``` Das `asChild`-Pattern erwartet, dass das Child Event-Handler (insb. `onPress`/`onClick`) korrekt mergt. TanStack-Router `<Link>` rendert ein nativen `<a>` und ruft beim Klick `preventDefault` + eigenen Navigations-Handler auf — der Popover-Close-Handler von Tamagui wird dabei vermutlich nicht ausgelöst. **Lösungsskizze (Optionen):** - Popover über `open`/`onOpenChange`-State steuern und `setOpen(false)` im Link-`onClick` aufrufen - Link in einen `Button` wrappen, der `router.navigate()` aufruft (dann funktioniert `Popover.Close asChild` mit Button-Trigger) - `Popover.Close` als äußeres Wrapper-Element verwenden (z.B. um den `MenuItem`-YStack statt um den Link) ## Bezug - Einführung: #336 (App-Shell-Refactor — Profil-Menü neu) - Begleit-Bug zu #348 (Settings-Layout)

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

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

admin-mrrm added the

type/bug

area/ui

area/web

labels

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

admin-mrrm referenced this issue from a commit

2026-05-20 06:45:22 +02:00

fix(#349): Profil-Menü-Popover schließt nach Item-Klick

admin-mrrm referenced this issue

2026-05-20 06:45:40 +02:00

fix(#349): Profil-Menü-Popover schließt nach Item-Klick #356

admin-mrrm referenced this issue

2026-05-20 07:18:57 +02:00

fix(#347): Web-Drawer als Side-Drawer statt Bottom-Sheet #357

admin-mrrm referenced this issue from a commit

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

Merge pull request 'fix(#349): Profil-Menü-Popover schließt nach Item-Klick' (#356) from fix/349-profile-menu-close 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:55 +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:08 +02:00

Collaborator

PM-Housekeeping: Fix wurde bereits am 2026-05-20 via PR #356 (Commit 7b30b04) auf main gemerged und ist seitdem in Prod ausgerollt. Issue wurde nur nicht automatisch geschlossen weil der Commit-Title fix(#349): statt Fixes #349 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 #356 (Commit `7b30b04`) auf `main` gemerged und ist seitdem in Prod ausgerollt. Issue wurde nur nicht automatisch geschlossen weil der Commit-Title `fix(#349):` statt `Fixes #349` 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:08 +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