-
Notifications
You must be signed in to change notification settings - Fork 87
Closed
Labels
a11yAccessibility issueAccessibility issuerefactorInternal improvementInternal improvementrequires new majorThis would be a breaking changeThis would be a breaking changev25To be done in V25To be done in V25vaadin-login
Description
Motivation
Currently, vaadin-login-form uses the following DOM structure internally:
web-components/packages/login/src/vaadin-login-form-wrapper.js
Lines 61 to 62 in ebf5367
| <section part="form"> | |
| <div part="form-title" role="heading" aria-level="${this.headingLevel}">${this.i18n.form.title}</div> |
This could be simplified in V25 by making vaadin-login-form-wrapper use role="region" instead.
Proposed solution
- Remove
<section>and setrole="region"on thevaadin-login-form-wrapperelement - Replace
part="form-title"with a slotted title and setaria-labelledbyon the form wrapper
Metadata
Metadata
Assignees
Labels
a11yAccessibility issueAccessibility issuerefactorInternal improvementInternal improvementrequires new majorThis would be a breaking changeThis would be a breaking changev25To be done in V25To be done in V25vaadin-login