Skip to content

7645 (fix) Forms cannot be submitted with focus on checkbox/radio#8219

Merged
deleonio merged 3 commits intopublic-ui:release/2from
aTTiCuZ-CodingPage:7645-fix-forms-cannot-be-submitted-with-focus-on-checkbox
Aug 6, 2025
Merged

7645 (fix) Forms cannot be submitted with focus on checkbox/radio#8219
deleonio merged 3 commits intopublic-ui:release/2from
aTTiCuZ-CodingPage:7645-fix-forms-cannot-be-submitted-with-focus-on-checkbox

Conversation

@aTTiCuZ-CodingPage
Copy link
Copy Markdown
Contributor

Ref.: #7645

Beschreibung
Dieser PR behebt ein Problem, bei dem ein Formular nicht mit der Enter-Taste abgesendet werden konnte, wenn der Fokus auf einer Checkbox lag. Bei der Radio-Komponente war dieses Problem nicht mehr vorhanden.

Änderungen

  • In der shadow.tsx der input-checkbox wurde dem -Element ein onKeyDown-Handler hinzugefügt.
  • Die neue Methode onKeyDown ruft intern die bereits bestehende Funktion propagateSubmitEventToForm auf.
  • Neue Testseite: form-submit.tsx unter packages/samples/react/src/scenarios/ hinzugefügt.
  • End-to-End-Test: form-submit.spec.ts mit Playwright unter packages/samples/react/e2e, der das beschriebene Verhalten überprüft.

Test

  • Manuelles Testen im Browser: scenarios -> form-submit
    • Beinhaltet 2 Forms (mit und ohne Submit-Button) mit Radio und Checkbox

E2E-Test mit Playwright (mit und ohne Submit-Button):
→ Checkbox im Fokus → Enter → Formular wird korrekt abgesendet
→ Radio im Fokus → Enter → Formular wird korrekt abgesendet

@anicyne anicyne linked an issue Aug 4, 2025 that may be closed by this pull request
4 tasks
Copy link
Copy Markdown
Contributor

@deleonio deleonio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @aTTiCuZ-CodingPage,

vielen Dank. Ich glaube, es gibt schon ein Formular, das den Submit für alle Eingabefelder mit Enter abbilden soll.

https://dev.kolibri.itz.in.bund.de/kolibri-demo/v3/#/scenarios/static-form

Bitte immer erst das Review in einer Version abwarten, bevor wir die Umsetzung in einer nächsten Version nachziehen.

@aTTiCuZ-CodingPage
Copy link
Copy Markdown
Contributor Author

aTTiCuZ-CodingPage commented Aug 5, 2025

Danke für dein Feedback @deleonio. Oh ja stimmt, es gibt dafür schon die static-form . Ich werde scenario/form-submit wieder entfernen und passe den e2e Test entsprechend an.

Alles kar. Ich werde künftig lieber auf die Review warten, bevor ich die nächste Version umsetze 👍

@deleonio deleonio merged commit ff770e4 into public-ui:release/2 Aug 6, 2025
8 checks passed
@publicuibot publicuibot bot locked and limited conversation to collaborators Aug 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Formulare lassen sich nicht mit Fokus auf Checkbox/Radio abschicken

2 participants