Skip to content
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

Dialog : use the locale property aria.close #7301

Closed
kyybo opened this issue Oct 3, 2024 · 0 comments · Fixed by #7302
Closed

Dialog : use the locale property aria.close #7301

kyybo opened this issue Oct 3, 2024 · 0 comments · Fixed by #7302
Assignees
Labels
Component: Accessibility Issue or pull request is related to WCAG or ARIA
Milestone

Comments

@kyybo
Copy link
Contributor

kyybo commented Oct 3, 2024

Describe the bug

Currently, the Dialog close button uses the local variable close and not aria.close as indicated in the documentation.

Reproducer

No response

System Information

"primereact": "10.8.3"
"react": "^18.2.0"

Steps to reproduce the behavior

No response

Expected behavior

No response

@kyybo kyybo added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Oct 3, 2024
melloware added a commit to melloware/primereact that referenced this issue Oct 3, 2024
@melloware melloware removed the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Oct 3, 2024
kyybo added a commit to kyybo/primereact that referenced this issue Oct 3, 2024
@melloware melloware self-assigned this Oct 3, 2024
@melloware melloware added the Component: Accessibility Issue or pull request is related to WCAG or ARIA label Oct 3, 2024
@melloware melloware added this to the 10.8.4 milestone Oct 3, 2024
@kyybo kyybo closed this as completed Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Accessibility Issue or pull request is related to WCAG or ARIA
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants