Conversation
|
Claude finished @tac0turtle's task in 1m 55s —— View job Code Review
Overall this is a clean visual restyle. The brand tokens, typography, and component styling are well-organized. A few issues below range from a real bug to minor nits. Bug: malformed
|
| Severity | Issue |
|---|---|
| Bug | og:description always outputs "false" |
| Accessibility | user-scalable=no blocks zoom for low-vision users |
| Correctness | Dark logo misconfigured (harmless now, breaks if dark mode re-enabled) |
| Tooling | package-lock.json committed alongside yarn.lock |
| Minor | Unused constants import in index.md |
| Minor | editLink repo path may be wrong |
| Nit | Inline styles and CSS variable underuse |
|
|
The latest Buf updates on your PR. Results from workflow CI / buf-check (pull_request).
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3210 +/- ##
==========================================
- Coverage 61.46% 61.45% -0.02%
==========================================
Files 120 120
Lines 12462 12470 +8
==========================================
+ Hits 7660 7663 +3
- Misses 3944 3947 +3
- Partials 858 860 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
julienrbrt
left a comment
There was a problem hiding this comment.
I haven't checked the content, but the UI looks so much better.
Overview
This pr ties and is blocked by the new landing page being launched