Skip to content

feat: add HasAriaDescription to field components - #9811

Merged
vursen merged 1 commit into
mainfrom
feat/has-aria-description
Aug 7, 2026
Merged

vursen merged 1 commit into
mainfrom
feat/has-aria-description

Conversation

@vursen

@vursen vursen commented Aug 3, 2026 •

Copy link
Copy Markdown
Contributor

This PR adds the HasAriaDescription mixin interface to all field components that implement HasAriaLabel, overriding setAriaDescribedBy / getAriaDescribedBy to map to the accessibleDescriptionRef property, which propagates the reference to the input element (the default implementation sets attributes on the host element instead). This follows the same pattern as accessibleName / accessibleNameRef.

DateTimePicker is left out because it doesn't implement HasAriaLabel, HasPrefix or HasSuffix.

Part of vaadin/web-components#11975

@vursen
vursen force-pushed the feat/has-aria-description branch 2 times, most recently from bb7b0a7 to 2498144 Compare August 5, 2026 13:49
Maps setAriaDescribedBy / getAriaDescribedBy to the accessibleDescriptionRef web component property, which propagates the reference to the input element.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@vursen
vursen force-pushed the feat/has-aria-description branch from 2498144 to 2e8d809 Compare August 5, 2026 13:55
@vursen
vursen marked this pull request as ready for review August 5, 2026 13:57
@vursen
vursen marked this pull request as draft August 5, 2026 13:58

@vaadin-review-bot vaadin-review-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

✅ Nothing to flag — the changes look good.

@sonarqubecloud

sonarqubecloud Bot commented Aug 5, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
29.6% Duplication on New Code (required ≤ 10%)

See analysis details on SonarQube Cloud

@vursen
vursen marked this pull request as ready for review August 6, 2026 07:57

@vaadin-review-bot vaadin-review-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

✅ Nothing to flag — the changes look good.

@vursen
vursen added this pull request to the merge queue Aug 7, 2026
Merged via the queue into main with commit 5ca4aba Aug 7, 2026
35 of 36 checks passed
@vursen
vursen deleted the feat/has-aria-description branch August 7, 2026 06:33
@vaadin-bot

Copy link
Copy Markdown
Collaborator

This ticket/PR has been released with Vaadin 25.3.0-alpha8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants