Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Hi! Thank you for opening this PR. Want me to review it? Based on the diff (81 lines, 5 files), I've queued these reviewers:
How this works
|
Bundle size comparisonBase commit: could not be determined; will be reported when the comparison runs Pending — |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
No unresolved blocking issues were identified.
Review effort: Lite
Findings: None
What changed in this PR
Updates homepage “Learn more” links with unique, visible-label-first accessible names and adds Playwright coverage.
Changes:
- Adds contextual
Learn more about <title>labels. - Removes redundant accessible-name props.
- Tests all affected homepage links.
| File | Description |
|---|---|
website/test/site-tests/Accessibility.spec.ts |
Adds coverage for homepage link names. |
website/src/components/home/serviceSection.tsx |
Updates service link names and props. |
website/src/components/home/partnersSection.tsx |
Updates partner link names and props. |
website/src/components/home/keyFeaturesSection.tsx |
Removes redundant props. |
website/src/components/home/keyFeatureCard.tsx |
Updates key-feature link names. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
🔗 No broken links found! ✅ Your attention to detail is admirable. linkcheck output |
Fleet Review — CleanNo issues found across the reviewer fleet for this run. |
Description
Updates homepage "Learn more" links to use unique accessible names in the form "Learn more about <title>". This keeps the visible label at the beginning of each accessible name while providing enough context to distinguish links for key features, services, and partners.
Removes redundant accessible-name props and adds Playwright coverage for every affected homepage link.
AB#83302
Reviewer Guidance
The review process is outlined in the pull request guidelines.