fix(a11y): add dynamic aria-label to copy button in hero-code.tsx#487
fix(a11y): add dynamic aria-label to copy button in hero-code.tsx#487Shiva210Jyoti wants to merge 1 commit into
Conversation
…loses piyushdotcomm#463 Signed-off-by: Shiva <jyotishiva2104@gmail.com>
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
👋 Thanks for opening a PR, @Shiva210Jyoti!Your PR has entered the 🚦 PR Review Pipeline.
What happens next
A pipeline status comment will appear below and update automatically as your PR progresses. While you wait
This comment is posted only once. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
WalkthroughThe PR adds a dynamic ChangesCopy Button Accessibility
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@piyushdotcomm could you please review this PR when you get a chance? I'm the assigned contributor for issue #463. Thank you! |
Summary
Adds a dynamic
aria-labelto the copy-to-clipboard button inmodules/home/hero-code.tsx.Type of change
Related issue
Closes #463
What changed
aria-label={copied ? "Copied to clipboard" : "Copy code"}to the copy buttonValidation
npm run lintContext
This is the same class of fix applied to other components via #283, #281, and #279. The homepage hero section was missed in those passes. This brings it to the same standard.
I am the assigned contributor for this issue (GSSoC'26).
Summary by CodeRabbit