Skip to content

Commit 59091ce

Browse files
tyler-daneclaude
andcommitted
fix(web): restore role=form on event forms
The styled-to-tailwind migration replaced <StyledEventForm role="form"> with a bare <form>, dropping the explicit ARIA role. A <form> without an accessible name exposes no "form" role, so getByRole("form") stopped matching and broke the event-form e2e flows (19 specs). Restore role="form" on both the timed and someday event forms. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 809861e commit 59091ce

2 files changed

Lines changed: 2 additions & 0 deletions

File tree

packages/web/src/views/Forms/EventForm/EventForm.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -455,6 +455,7 @@ export const EventForm: React.FC<Omit<FormProps, "category">> = memo(
455455
return (
456456
<form
457457
{...props}
458+
role="form"
458459
className="c-event-form"
459460
name={ID_EVENT_FORM}
460461
onMouseUp={() => {

packages/web/src/views/Forms/SomedayEventForm/SomedayEventForm.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -161,6 +161,7 @@ export const SomedayEventForm: React.FC<FormProps> = ({
161161
return (
162162
<form
163163
{...props}
164+
role="form"
164165
className="c-event-form"
165166
name={ID_SOMEDAY_EVENT_FORM}
166167
onClick={stopPropagation}

0 commit comments

Comments
 (0)