sffs-tt-basel/sffs-tt-issues
2
0
Fork 0
Code Issues 114 Pull requests Projects Releases Packages Wiki Activity Actions

Rückfragen als Browser-Dialog (window.confirm/prompt) statt als eigener Dialog #96

New issue
Closed
opened 2026-10-14 20:10:36 +00:00 by claude · 1 comment
claude commented 2026-10-14 20:10:36 +00:00
Collaborator
Copy link

Was falsch ist

Vier Stellen fragen über die nativen Browser-Dialoge nach, statt über die Dialoge der App. Das widerspricht der Regel in CLAUDE.md (Bestätigungen als AlertDialog, Eingaben in einem Formular-Dialog): Die Browser-Fenster sehen je nach Browser anders aus, lassen sich nicht gestalten, haben kleine Schrift und englische Knöpfe („OK“/„Cancel“) – für ältere Nutzer schlechter lesbar.

Stelle Was
portal/src/components/captain/ProposalAnswerView.tsx:45 Captain: „Wirklich alle Terminvorschläge ablehnen?“ (window.confirm)
content/src/pages/EventEditDialog.tsx:146 Redaktion: „Event … wirklich löschen?“ (window.confirm)
content/src/pages/EventManager.tsx:86 Redaktion: „Name der neuen Saison (z.B. 2025/2026)“ (window.prompt)
content/src/components/RichTextEditor.tsx:52 Editor: „URL“ für einen Link (window.prompt)

Am 14.10.2026 vollständig gesucht (window.confirm|prompt|alert in portal und content, ohne Tests).

Was es werden soll

Bestätigungen als AlertDialog wie bei „Termin aufheben“ oder „Begegnung löschen“, mit Paarung bzw. Namen und deutschen Knöpfen. Eingaben (Saisonname, Link-Adresse) als kleiner Dialog mit Feld und Inline-Prüfung. Die E2E-Tests, die heute den Browser-Dialog bestätigen (captain-nebenwege.spec.ts, cms-saison-archive.spec.ts, cms-redaktion.spec.ts), ziehen mit.

Gefunden beim Nachziehen der Captain-E2E (ADR 0026).

## Was falsch ist Vier Stellen fragen über die nativen Browser-Dialoge nach, statt über die Dialoge der App. Das widerspricht der Regel in CLAUDE.md (Bestätigungen als AlertDialog, Eingaben in einem Formular-Dialog): Die Browser-Fenster sehen je nach Browser anders aus, lassen sich nicht gestalten, haben kleine Schrift und englische Knöpfe („OK“/„Cancel“) – für ältere Nutzer schlechter lesbar. | Stelle | Was | |---|---| | `portal/src/components/captain/ProposalAnswerView.tsx:45` | Captain: „Wirklich alle Terminvorschläge ablehnen?“ (`window.confirm`) | | `content/src/pages/EventEditDialog.tsx:146` | Redaktion: „Event … wirklich löschen?“ (`window.confirm`) | | `content/src/pages/EventManager.tsx:86` | Redaktion: „Name der neuen Saison (z.B. 2025/2026)“ (`window.prompt`) | | `content/src/components/RichTextEditor.tsx:52` | Editor: „URL“ für einen Link (`window.prompt`) | Am 14.10.2026 vollständig gesucht (`window.confirm|prompt|alert` in portal und content, ohne Tests). ## Was es werden soll Bestätigungen als AlertDialog wie bei „Termin aufheben“ oder „Begegnung löschen“, mit Paarung bzw. Namen und deutschen Knöpfen. Eingaben (Saisonname, Link-Adresse) als kleiner Dialog mit Feld und Inline-Prüfung. Die E2E-Tests, die heute den Browser-Dialog bestätigen (`captain-nebenwege.spec.ts`, `cms-saison-archive.spec.ts`, `cms-redaktion.spec.ts`), ziehen mit. Gefunden beim Nachziehen der Captain-E2E (ADR 0026).
claude commented 2026-10-18 14:22:24 +00:00
Author
Collaborator
Copy link

Seit 18.10. in main (544b0d7, Code-Stand 936723a, läuft so auf Staging). Kein Browser-Fenster mehr, alle vier Rückfragen sind Dialoge der App.

Prüfschritte auf Staging:

  1. Content → Redaktion → Events → Neue Saison: 2027-28 eingeben → Anlegen → rote Meldung am Feld „Bitte im Format 2027/2028 eingeben“. Eine schon vorhandene Saison eingeben → „Die Saison … gibt es schon“. Abbrechen und neu öffnen → Feld leer.
  2. Content → Redaktion → Events → Bearbeiten → „Event löschen“: Dialog „Event wirklich löschen?“ mit dem Namen des Events; Abbrechen lässt alles stehen. (Der Löschdialog in der Liste zeigt denselben Text.)
  3. Content → Redaktion → Artikel → Text markieren → Knopf „Link“: Dialog „Link einfügen“, vorbelegt mit https://. www.beispiel.ch → Meldung am Feld; https://beispiel.ch → Übernehmen setzt den Link. Cursor in den Link, Knopf „Link“ → „Link bearbeiten“ mit eigenem Knopf „Link entfernen“. Enter im Dialog speichert nicht den ganzen Artikel.
  4. Portal als Gast-Captain → Begegnung mit Terminvorschlägen → „Alle ablehnen“: Dialog nennt die Begegnung, die Zahl der Vorschläge und dass das Heimteam eine E-Mail bekommt; erst „Alle ablehnen“ im Dialog lehnt ab.

Am Handy (schmal) sind alle Dialoge bedienbar. Aus dem Review neu: #106 (Doppelklick auf Bestätigen, betrifft alle Bestätigungsdialoge, älter als #96).

Seit 18.10. in `main` (`544b0d7`, Code-Stand `936723a`, läuft so auf Staging). Kein Browser-Fenster mehr, alle vier Rückfragen sind Dialoge der App. **Prüfschritte auf Staging:** 1. **Content → Redaktion → Events → Neue Saison:** `2027-28` eingeben → Anlegen → rote Meldung am Feld „Bitte im Format 2027/2028 eingeben“. Eine schon vorhandene Saison eingeben → „Die Saison … gibt es schon“. Abbrechen und neu öffnen → Feld leer. 2. **Content → Redaktion → Events → Bearbeiten → „Event löschen“:** Dialog „Event wirklich löschen?“ mit dem Namen des Events; Abbrechen lässt alles stehen. (Der Löschdialog in der Liste zeigt denselben Text.) 3. **Content → Redaktion → Artikel → Text markieren → Knopf „Link“:** Dialog „Link einfügen“, vorbelegt mit `https://`. `www.beispiel.ch` → Meldung am Feld; `https://beispiel.ch` → Übernehmen setzt den Link. Cursor in den Link, Knopf „Link“ → „Link bearbeiten“ mit eigenem Knopf „Link entfernen“. Enter im Dialog speichert nicht den ganzen Artikel. 4. **Portal als Gast-Captain → Begegnung mit Terminvorschlägen → „Alle ablehnen“:** Dialog nennt die Begegnung, die Zahl der Vorschläge und dass das Heimteam eine E-Mail bekommt; erst „Alle ablehnen“ im Dialog lehnt ab. Am Handy (schmal) sind alle Dialoge bedienbar. Aus dem Review neu: #106 (Doppelklick auf Bestätigen, betrifft alle Bestätigungsdialoge, älter als #96).
olli closed this issue 2026-10-18 14:29:12 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
No results found.
Labels
Clear labels   
Kind
Bug

  
Kind
Enhancement

  
Kind
Maintenance
Umbau, Aufraeumen, Tests, Werkzeuge, Absicherung - ohne fachliche Aenderung

  
Priority
P1
Vor der Testphase auf Preview: falsche Rechte, falsche Regel im Captain-Ablauf oder stilles Scheitern

  
Priority
P2
Vor dem Prod-Rollout: im laufenden Betrieb noetig, teils mit festem Termin

  
Priority
P3
Wenn Zeit ist: Darstellung, Komfort, Hygiene

  
Status
In Progress

  
Status
Need More Info

  
Status
Zur Abnahme
In main gemergt und auf Staging; wartet auf Abnahme (Petra bzw. Olli), danach schliessen

  
Änderungen Wettspielreglement

Fehler oder Unklarheit im Wettspielreglement, die der Vorstand im Dokument berichtigen muss

No labels
Kind
Bug
Kind
Enhancement
Kind
Maintenance
Priority
P1
Priority
P2
Priority
P3
Status
In Progress
Status
Need More Info
Status
Zur Abnahme
Änderungen Wettspielreglement
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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
sffs-tt-basel/sffs-tt-issues#96
No description provided.