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

Addressed scrolling accessibility issue on homepage #3656

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tkakar
Copy link
Collaborator

@tkakar tkakar commented Dec 19, 2024

Summary

Fixes the accessibility issue Ensure elements that have scrollable content are accessible by keyboard on the Home Page, particularly on the chart's legend. This was not included in the report, but I found the issue during other fixes.

Design Documentation/Original Tickets

CAT-1083

Testing

Manually tested using the axe DevTool

Screenshots/Video

Include screenshots or video demonstrating any significant visual or behavioral changes.

Checklist

  • Code follows the project's coding standards
    • Lint checks pass locally
    • New CHANGELOG-your-feature-name-here.md is present in the root directory, describing the change(s) in full sentences.
  • Unit tests covering the new feature have been added
  • All existing tests pass
  • Any relevant documentation in JIRA/Confluence has been updated to reflect the new feature
  • Any new functionalities have appropriate analytics functionalities added

Additional Notes

Please specify any additional information or context relevant to this PR.

@tkakar tkakar changed the title Addressed issue on homepage Addressed scrolling accessibility issue on homepage Dec 19, 2024
@tkakar tkakar requested a review from NickAkhmetov December 20, 2024 14:43
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.

1 participant