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

Merged
admin-mrrm merged 1 commit from fix/349-profile-menu-close into main 2026-05-20 07:40:48 +02:00
Owner

Behebt #349.

Ursache

Popover.Close asChild um TanStack-Router <Link> funktionierte nicht — der Link ruft preventDefault + eigenen Navigations-Handler auf, der von Tamagui injizierte Close-Handler wird nicht ausgeführt.

Fix

Popover wird über open/onOpenChange State kontrolliert. Items rufen setOpen(false) im onClick/onPress auf. Popover.Close-Wrapper entfallen.

Tests

Neuer Spec profile-menu.spec.tsx mit 2 Tests:

  • Klick auf „Einstellungen" schließt das Popover
  • Klick auf „Konto" schließt das Popover

Vorher rot, nach Fix grün. Gesamte Web-Suite (67 Tests) passt durch.

Behebt #349. ## Ursache `Popover.Close asChild` um TanStack-Router `<Link>` funktionierte nicht — der Link ruft `preventDefault` + eigenen Navigations-Handler auf, der von Tamagui injizierte Close-Handler wird nicht ausgeführt. ## Fix Popover wird über `open`/`onOpenChange` State kontrolliert. Items rufen `setOpen(false)` im `onClick`/`onPress` auf. `Popover.Close`-Wrapper entfallen. ## Tests Neuer Spec `profile-menu.spec.tsx` mit 2 Tests: - Klick auf „Einstellungen" schließt das Popover - Klick auf „Konto" schließt das Popover Vorher rot, nach Fix grün. Gesamte Web-Suite (67 Tests) passt durch.
fix(#349): Profil-Menü-Popover schließt nach Item-Klick
All checks were successful
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is passing
7b30b043f7
Tamagui Popover.Close mit TanStack-Router Link funktionierte nicht, weil
der Link-Click-Handler den Close-Handler nicht durchreicht. Popover jetzt
über open/setOpen kontrolliert, Links rufen setOpen(false) im onClick auf.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
admin-mrrm/mrrmlabapp!356
No description provided.