fix(mobile): ProfileMenu von Sheet auf Popover umgestellt #520

Merged
admin-mrrm merged 2 commits from fix/mobile-profile-menu-popover into main 2026-07-25 19:20:58 +02:00
Owner

Closes #519

Ursache

Die Mobile-ProfileMenu-Komponente nutzte Tamagui's Sheet (Bottom-Sheet-Modal) statt Popover wie die Web-Version — dadurch öffnete sich das Menü vom unteren Bildschirmrand statt als Dropdown unter dem Avatar.

Fix

  • Sheet/Sheet.Overlay/Sheet.Frame/Sheet.Handle durch Popover/Popover.Trigger/Popover.Content/Popover.Arrow ersetzt, analog zu apps/web/src/components/profile-menu.tsx
  • placement="bottom-end" + allowFlip ergänzt für Web-Parität
  • Popover.Trigger korrekt als Kind von Popover verschachtelt (vorherige Zwischenversion hatte Trigger und Content fälschlich als Geschwister statt beide innerhalb von Popover)
  • Sheet-spezifische Props (modal, snapPointsMode, dismissOnSnapToBottom) entfernt, die auf Popover nicht existieren (TS2322)

Test

Kein neuer Test ergänzt — reine Presentational-Komponente ohne Business-Logik, UI-Struktur wird durch Typecheck abgesichert. typecheck, lint und alle 203 Unit-Tests grün. Manuell auf echtem Gerät verifiziert: Popover öffnet sich korrekt unter dem Avatar.

Closes #519 ## Ursache Die Mobile-`ProfileMenu`-Komponente nutzte Tamagui's `Sheet` (Bottom-Sheet-Modal) statt `Popover` wie die Web-Version — dadurch öffnete sich das Menü vom unteren Bildschirmrand statt als Dropdown unter dem Avatar. ## Fix - `Sheet`/`Sheet.Overlay`/`Sheet.Frame`/`Sheet.Handle` durch `Popover`/`Popover.Trigger`/`Popover.Content`/`Popover.Arrow` ersetzt, analog zu `apps/web/src/components/profile-menu.tsx` - `placement="bottom-end"` + `allowFlip` ergänzt für Web-Parität - `Popover.Trigger` korrekt als Kind von `Popover` verschachtelt (vorherige Zwischenversion hatte Trigger und Content fälschlich als Geschwister statt beide innerhalb von `Popover`) - Sheet-spezifische Props (`modal`, `snapPointsMode`, `dismissOnSnapToBottom`) entfernt, die auf `Popover` nicht existieren (TS2322) ## Test Kein neuer Test ergänzt — reine Presentational-Komponente ohne Business-Logik, UI-Struktur wird durch Typecheck abgesichert. `typecheck`, `lint` und alle 203 Unit-Tests grün. Manuell auf echtem Gerät verifiziert: Popover öffnet sich korrekt unter dem Avatar.
fix(mobile): ProfileMenu von Sheet auf Popover umgestellt
All checks were successful
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is passing
53a36512c7
Closes #519
Merge branch 'main' into fix/mobile-profile-menu-popover
All checks were successful
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is passing
48439b34d7
admin-mrrm scheduled this pull request to auto merge when all checks succeed 2026-07-25 19:15:03 +02:00
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!520
No description provided.