Conversation
Agent-Logs-Url: https://github.com/aleexwong/trainpace/sessions/39e9789e-4b18-411f-96a0-ca4f3950a698 Co-authored-by: aleexwong <65264501+aleexwong@users.noreply.github.com>
…nner slogan badge Agent-Logs-Url: https://github.com/aleexwong/trainpace/sessions/517c371d-bf98-4c76-9ceb-dd765f6ac950 Co-authored-by: aleexwong <65264501+aleexwong@users.noreply.github.com>
Agent-Logs-Url: https://github.com/aleexwong/trainpace/sessions/517c371d-bf98-4c76-9ceb-dd765f6ac950 Co-authored-by: aleexwong <65264501+aleexwong@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
✅ Deploy Preview for trainpace ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Not up to standards ⛔🔴 Issues
|
| Category | Results |
|---|---|
| ErrorProne | 2 medium |
🟢 Metrics 0 complexity · 0 duplication
Metric Results Complexity 0 Duplication 0
TIP This summary will be updated as you push new changes. Give us feedback
There was a problem hiding this comment.
Pull request overview
Updates the Landing page homepage dashboard/hero content to better promote race-course browsing and adjust CTA/copy messaging.
Changes:
- Updated Hero badge + primary CTA copy (“Get Started”).
- Reworked the Hero “dashboard mockup” into a clickable preview that navigates to
/raceand updated its content/iconography. - Updated a PainPoints entry and aligned CTA button copy (“Get Started”).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <div | ||
| className="relative rounded-2xl bg-white shadow-2xl border border-slate-200 overflow-hidden cursor-pointer hover:shadow-emerald-200 hover:border-emerald-300 transition-all duration-200 group" | ||
| onClick={() => navigate("/race")} | ||
| onKeyDown={(e) => (e.key === "Enter" || e.key === " ") && navigate("/race")} | ||
| role="link" |
There was a problem hiding this comment.
This clickable card is implemented as a
role="link" and custom keyboard handling. For better accessibility/semantics (and to get correct keyboard behavior + SEO for free), consider using a real <Link to="/race"> (or an <a href>), rather than a generic div with ARIA role.| <div | ||
| className="relative rounded-2xl bg-white shadow-2xl border border-slate-200 overflow-hidden cursor-pointer hover:shadow-emerald-200 hover:border-emerald-300 transition-all duration-200 group" | ||
| onClick={() => navigate("/race")} | ||
| onKeyDown={(e) => (e.key === "Enter" || e.key === " ") && navigate("/race")} |
There was a problem hiding this comment.
onKeyDown triggers navigation on Space but doesn’t call preventDefault(), so Space can also scroll the page while the element is focused. Also, Space activation is expected for role="button" rather than role="link" (links generally activate on Enter). Align the role/keyboard interaction and prevent the default Space behavior when handling it.
| onKeyDown={(e) => (e.key === "Enter" || e.key === " ") && navigate("/race")} | |
| onKeyDown={(e) => e.key === "Enter" && navigate("/race")} |
| problem: "Coaches are expensive ($150+/mo).", | ||
| solution: "TrainPace is 100% free, forever.", | ||
| problem: "I want to know what pace I should be going per km.", | ||
| solution: "Get your exact per-km pace zones from any recent race time.", |
There was a problem hiding this comment.
Copy consistency: this problem statement uses “per km” while the solution right below uses “per-km”. Consider standardizing the phrasing (either both “per km” or both “per-km”) to avoid inconsistent UI copy.
| solution: "Get your exact per-km pace zones from any recent race time.", | |
| solution: "Get your exact per km pace zones from any recent race time.", |
|
@copilot apply changes based on the comments in this thread |
No description provided.