-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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: Add missing aria labels and roles to buttons, links, tables #11577
Fix: Add missing aria labels and roles to buttons, links, tables #11577
Conversation
DryRun Security SummaryThe pull request focuses on improving web accessibility across multiple Dojo application pages by adding ARIA attributes to buttons and dropdown menus to enhance screen reader compatibility and user experience. Expand for full summarySummary: The code changes in this pull request focus on improving the accessibility and usability of various pages and features in the Dojo application. The changes primarily involve adding From a security perspective, the changes do not introduce any obvious vulnerabilities. The code appears to follow secure coding practices, such as using Django's built-in template engine, implementing access control mechanisms, and avoiding direct rendering of user-supplied input. However, it is important to review the entire codebase and consider the broader security implications of the application, including input validation, authentication, authorization, error handling, and the use of third-party libraries and dependencies. Files Changed:
Code AnalysisWe ran
Overall Riskiness🔴 Risk threshold exceeded. We've notified @mtesauro, @grendel513. |
bc24ed4
to
59d7c93
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
@littlesvensson Thanks for this PR. We've had accessibility lint running for years but I don't know much more then what it notices so thanks for sharing your knowledge on the right thing to do for screen readers. It's MUCH appreciated. 🎉 |
Description
Fixes #11576
Visually there is no change, only some missing accessibility attributes added