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

Fix #7110 - SpeedDial : Fix aria-controls values #7111

Merged
merged 4 commits into from
Sep 4, 2024

Conversation

kyybo
Copy link
Contributor

@kyybo kyybo commented Sep 4, 2024

"The aria-controls attribute identifies the element (or elements) whose contents or presence are controlled by the element on which the attribute is set" (--> https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes/aria-controls)

The aria-controls attribute should only be placed on the SpeedDial trigger button. And its value must be the identifier or list of identifiers of its items.

Fix #7110

"The aria-controls attribute identifies the element (or elements) whose contents or presence are controlled by the element on which the attribute is set" (--> https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes/aria-controls)

The aria-controls attribute should only be placed on the SpeedDial trigger button. And its value must be the identifier or list of identifiers of its items.
Copy link

vercel bot commented Sep 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
primereact ⬜️ Ignored (Inspect) Visit Preview Sep 4, 2024 1:14pm
primereact-v9 ⬜️ Ignored (Inspect) Visit Preview Sep 4, 2024 1:14pm

@melloware
Copy link
Member

can you run npm run format to format the code please.

a comma was missing
@melloware
Copy link
Member

Its still failing the format check npm run format:check it looks like it formatted too much now? It should just fix your file if you run npm run format from the root project.

Wrong prettier config in last commit (822633a), quote were replace by double quote
@kyybo
Copy link
Contributor Author

kyybo commented Sep 4, 2024

Sorry this is my first PR, I'm not familiar with all the procedures yet.
As I edited the file directly from github I couldn't run the npm run format command. But I'm gonna make it 😅

@melloware
Copy link
Member

No worries that is what reviews are for!

@melloware
Copy link
Member

ok now run npm run lint:fix it has some lint errors.

@melloware melloware merged commit 26240b5 into primefaces:master Sep 4, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SpeedDial : Fix aria-controls values
2 participants