Skip to content

feat(ConfirmDialog): add setter/getter for aria-describedby #4913

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
May 15, 2023

Conversation

DiegoCardoso
Copy link
Contributor

Description

Add API to override the default aria-describedby value set by the web component.

Blocked by vaadin/web-components#5728
Blocked by vaadin/web-components#5730
Fixes vaadin/web-components#4684

Type of change

  • Bugfix
  • Feature

@DiegoCardoso DiegoCardoso requested a review from web-padawan April 3, 2023 13:29
@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@DiegoCardoso DiegoCardoso requested a review from vursen May 11, 2023 11:06
DiegoCardoso and others added 2 commits May 15, 2023 09:53
…rc/main/java/com/vaadin/flow/component/confirmdialog/ConfirmDialog.java

Co-authored-by: Sergey Vinogradov <mr.vursen@gmail.com>
vursen
vursen previously approved these changes May 15, 2023
@vursen vursen dismissed their stale review May 15, 2023 07:32

Made approval by mistake

Co-authored-by: Sergey Vinogradov <mr.vursen@gmail.com>
DiegoCardoso and others added 3 commits May 15, 2023 10:46
…rc/main/java/com/vaadin/flow/component/confirmdialog/ConfirmDialog.java

Co-authored-by: Sergey Vinogradov <mr.vursen@gmail.com>
@DiegoCardoso DiegoCardoso enabled auto-merge (squash) May 15, 2023 08:58
@DiegoCardoso DiegoCardoso disabled auto-merge May 15, 2023 09:05
@DiegoCardoso DiegoCardoso enabled auto-merge (squash) May 15, 2023 14:09
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@DiegoCardoso DiegoCardoso merged commit f53f9a9 into main May 15, 2023
@DiegoCardoso DiegoCardoso deleted the feat/confirm-dialog-describedby-api branch May 15, 2023 14:18
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.1.0.

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.

[confirm-dialog] Some ATs do not announce the dialog's message
3 participants