Skip to content

feat(frontend): add accessible labels to frontend shell#1621

Open
kshah123919 wants to merge 2 commits into
imDarshanGK:mainfrom
kshah123919:feat/add-accessible-labels-frontend-shell-v2
Open

feat(frontend): add accessible labels to frontend shell#1621
kshah123919 wants to merge 2 commits into
imDarshanGK:mainfrom
kshah123919:feat/add-accessible-labels-frontend-shell-v2

Conversation

@kshah123919

Copy link
Copy Markdown
Contributor

Description

Improved the accessibility of the frontend UI by adding descriptive aria-label attributes to interactive elements that were missing accessible labels. These changes enhance screen reader support and improve the overall accessibility of the frontend without affecting existing functionality or the user interface.

Related Issue

Fixes #1586

Type of change

  • Bug fix
  • New feature / enhancement
  • Documentation update
  • Test addition
  • Refactor

Checklist

  • I have read CONTRIBUTING.md
  • My branch is up to date with main
  • I have run pytest -v and all tests pass
  • I have not introduced duplicate issues or features
  • My PR title follows the format: feat/fix/docs/test: short description
  • I have added tests for new features (Level 2 and 3 issues)
  • No hardcoded secrets or API keys in my code
  • This PR is linked to a GSSoC 2026 issue

Screenshots (if frontend change)

No visual UI changes. This PR improves accessibility by adding accessible labels for screen readers.

Test evidence

Not applicable. The changes are limited to HTML accessibility attributes and do not modify application logic.

@kshah123919
kshah123919 requested a review from imDarshanGK as a code owner July 7, 2026 05:46
@kshah123919

Copy link
Copy Markdown
Contributor Author

Hi @imDarshanGK ,

I've completed the changes for this issue and opened a PR. I added the accessibility improvements by providing descriptive aria-label attributes to the relevant frontend UI elements while keeping the existing functionality unchanged.

Whenever you have time, could you please review the PR? If there are any changes or improvements you'd like me to make, I'd be happy to update them.
Thank you!

@github-actions

Copy link
Copy Markdown

👋 This PR has had no activity for 7 days.

Please push updates or comment if you still need more time.

Inactive PRs may be closed automatically after 7 more days.

@github-actions github-actions Bot added the stale label Jul 15, 2026
@kshah123919

Copy link
Copy Markdown
Contributor Author

Hi @imDarshanGK !
Just a gentle reminder regarding my PR. It's been 7 days since I submitted it. Could you please review and merge it when you have time? If any changes are needed, I'll be happy to update them.

@imDarshanGK imDarshanGK left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kshah123919 add video demo

@github-actions github-actions Bot removed the stale label Jul 16, 2026
@kshah123919

Copy link
Copy Markdown
Contributor Author

Demo Video: Demonstrates the application running and highlights the added aria-label attributes in the modified HTML to show the accessibility improvements introduced by this PR.

16.07.2026_09.19.53_REC.mp4

@kshah123919

Copy link
Copy Markdown
Contributor Author

Hi @imDarshanGK !
I've added the demo video to the PR as requested. Could you please take a look whenever you have time? If there are any additional changes or improvements needed, I'd be happy to update them.

@kshah123919
kshah123919 requested a review from imDarshanGK July 16, 2026 03:53

@imDarshanGK imDarshanGK left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kshah123919 Please upload a video demo of the feature in the running application. The current submission is a code demo, not a feature demo as requested.

@kshah123919

Copy link
Copy Markdown
Contributor Author

Hi @imDarshanGK !
I've updated the PR with a feature demo. Since this change adds aria-label attributes, I recorded the running application and also demonstrated the accessibility information in the browser's Accessibility panel, where the accessible name generated by the aria-label can be verified. Kindly have a look.

18.07.2026_15.39.52_REC.mp4

@kshah123919
kshah123919 requested a review from imDarshanGK July 18, 2026 10:14
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.

Add accessible labels to frontend shell (v2)

2 participants