refactor(frontend): ersetze Inline-Buttons durch ActionMenu in DataTables #241

Merged

niboer merged 2 commits from issue-219 into main

2026-07-05 19:53:25 +00:00

niboer commented

2026-07-05 19:40:28 +00:00

Owner

Schliesst #219

Ersetzt <Button>-Elemente in Actions-Spalten von 5 DataTables durch <ActionMenu> (drei-Punkte-Menue).

Betroffene Dateien

groups/+page.svelte: Bearbeiten/Loeschen → ActionMenu
groups/[groupId]/+page.svelte: Genehmigen/Ablehnen (pending requests) → ActionMenu
templates/+page.svelte: Bearbeiten (href) / Loeschen → ActionMenu
events/[eventId]/+page.svelte: Rollen-Bearbeiten/Loeschen → ActionMenu
deletion-requests/+page.svelte: Genehmigen/Ablehnen → ActionMenu

Test-Aenderungen

Unit-Tests: Import- und Struktur-Tests auf ActionMenu aktualisiert (917 passed)
E2E-Test: btn-approve-join → action-menu-toggle + action-menu-form-genehmigen
Lint & testid-Check: OK

Schliesst #219 Ersetzt `<Button>`-Elemente in Actions-Spalten von 5 DataTables durch `<ActionMenu>` (drei-Punkte-Menue). ## Betroffene Dateien - **groups/+page.svelte**: Bearbeiten/Loeschen → ActionMenu - **groups/[groupId]/+page.svelte**: Genehmigen/Ablehnen (pending requests) → ActionMenu - **templates/+page.svelte**: Bearbeiten (href) / Loeschen → ActionMenu - **events/[eventId]/+page.svelte**: Rollen-Bearbeiten/Loeschen → ActionMenu - **deletion-requests/+page.svelte**: Genehmigen/Ablehnen → ActionMenu ## Test-Aenderungen - Unit-Tests: Import- und Struktur-Tests auf ActionMenu aktualisiert (917 passed) - E2E-Test: `btn-approve-join` → `action-menu-toggle` + `action-menu-form-genehmigen` - Lint & testid-Check: OK

niboer added 2 commits

2026-07-05 19:40:28 +00:00

refactor(frontend): ersetze Inline-Buttons durch ActionMenu in DataTables aba4dd310c

closes #219

Aktuelles: Fuenf DataTables verwenden noch inline <Button>-Elemente in der Actions-Spalte (groups, groups/[groupId] pending requests, templates, events/[eventId] rollen, deletion-requests)

Neu: Alle fuenf Dateien nutzen <ActionMenu> mit konsistentem Pattern
- groups/+page.svelte: Bearbeiten/Loeschen via onClick
- groups/[groupId]/+page.svelte: Genehmigen/Ablehnen via type:'submit'
- templates/+page.svelte: Bearbeiten via href, Loeschen via onClick
- events/[eventId]/+page.svelte: Rollen-Bearbeiten/Loeschen via onClick
- deletion-requests/+page.svelte: Genehmigen/Ablehnen via onClick

Unit-Tests: 917 passed, Lint: OK
E2E: btn-approve-join -> action-menu-form-genehmigen

Wichtige Aenderung:
- Alte data-testid-Attribute entfallen (werden durch Auto-generierte ActionMenu-testids ersetzt)
- Nur btn-approve-join wurde in E2E-Tests referenziert

fix(e2e): fehlendes action-menu-toggle vor dem Genehmigen-Klick

CI Backend / lint-backend (pull_request) Successful in 50s

Details

CI Frontend / lint-frontend (pull_request) Successful in 1m11s

Details

CI Meta / conventional-commit (pull_request) Successful in 2s

Details

E2E / e2e (pull_request) Successful in 3m22s

Details

CI Backend / test-backend (pull_request) Successful in 1m43s

Details

CI Frontend / test-frontend (pull_request) Successful in 42s

Details

3708a4b54a

refs #219

Aktuelles: E2E-Test klickt action-menu-form-genehmigen direkt an, aber der ActionMenu-Dropdown ist nicht geoeffnet -> Element nicht sichtbar -> Test timeout

Neu: Vor dem Klick auf das Formular wird zunaechst der action-menu-toggle-Button in der korrekten Zeile geklickt (Pattern konsistent zu bestehendem Mitglied-entfernen-Test)

niboer merged commit 21b273fadf into main

2026-07-05 19:53:25 +00:00

niboer deleted branch issue-219

2026-07-05 19:53:25 +00:00

niboer referenced this pull request from a commit

2026-07-05 19:53:25 +00:00

refactor(frontend): ersetze Inline-Buttons durch ActionMenu in DataTables (#241)