Skip to content

fix: label result action buttons#1613

Open
mittalsonal wants to merge 2 commits into
imDarshanGK:mainfrom
mittalsonal:1601-accessible-security-labels
Open

fix: label result action buttons#1613
mittalsonal wants to merge 2 commits into
imDarshanGK:mainfrom
mittalsonal:1601-accessible-security-labels

Conversation

@mittalsonal

Copy link
Copy Markdown

Summary

  • add accessible names for the result action icon buttons
  • cover favorites, download, PDF export, and Markdown export actions
  • keep the visible UI and existing button behavior unchanged

Fixes #1601

Validation

  • Parsed frontend/index.html with Python html.parser
  • Ran git diff --check -- frontend/index.html

Checklist

  • I have read the contributing guidelines
  • My PR is small and focused
  • No hardcoded secrets or API keys in my code
  • This PR is linked to a GSSoC 2026 issue

@mittalsonal
mittalsonal requested a review from imDarshanGK as a code owner July 2, 2026 18:21
@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 10, 2026
@mittalsonal

Copy link
Copy Markdown
Author

Still active and ready for review. There are no maintainer-requested changes on this PR from my side right now, but I?m happy to address any feedback quickly.

@github-actions github-actions Bot removed the stale label Jul 12, 2026
@mittalsonal
mittalsonal force-pushed the 1601-accessible-security-labels branch from b2e1116 to 1d654b9 Compare July 15, 2026 19:06

@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.

@mittalsonal add video demo

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 security utils

2 participants