Add plain-language context to home page#91
Conversation
bandaid on the client submission form
* feat: add Vercel Analytics integration * fix: harden server analytics tracking * fix: classify consultation submission errors
* feat: improve form error message handling * fix: wire services field accessibility * docs: add PR review workflow documentation
* CR #131: Update cohort join page to link to Portal join * CR #131: Update cohort join page to link to Portal join * CR #131: Remove dependency lockfile churn --------- Co-authored-by: Prism Codex <prism-codex@users.noreply.github.com>
* CR #133: Add Join navigation link on website root page * CR #133: Add Join navigation link on website root page * Revert "CR #133: Add Join navigation link on website root page" This reverts commit 5bc6eb0. --------- Co-authored-by: Prism Codex <prism-codex@users.noreply.github.com>
* adds ai solutions services * removes contact page * more add
Co-authored-by: Prism Codex <prism-codex@users.noreply.github.com>
Co-authored-by: Prism Codex <prism-codex@users.noreply.github.com>
|
Deployment failed with the following error: View Documentation: https://vercel.com/docs/accounts/team-members-and-roles |
📝 WalkthroughWalkthroughUpdated homepage labels, marketing copy, services layout, client copy, and hero tracking. The “since 2019” text now navigates to the story section through a tracked in-page link. ChangesHomepage refresh
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Hero: plain-language framing for the mercenary tagline, "since 2019" linked to the Our Story section, dashes replaced with other punctuation throughout. Services: two-column layout (Arsenal of Expertise / Looking for AI Solutions) filling the full section width, matching the col-span-4/8/6 pattern used in MercenariesSection. AI heading now matches the main heading style. CTA button sized to 4 of 12 grid columns with spacing matching the MercenariesSection template. Background illustration moved to the left. Mercenaries / Portfolio / Hire Us / Testimonials: small eyebrow labels above each section heading for consistency. Case studies: DAOhaus description trimmed, em dash removed. Our Story: em dash replaced with a comma in the Join the Guild copy. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2631f75 to
0192024
Compare
Tailwind CSS renamed several sizing utilities in v3.3 (flex-shrink-* -> shrink-*, flex-grow-* -> grow-*) and kept the old names only as deprecated aliases for backward compatibility. flex-shrink-0 still works today, but it's the legacy spelling — shrink-0 is the current, documented utility and is what new code in this repo should use going forward. This is a pure rename with no visual or behavioral change: shrink-0 compiles to the same shrink: 0 CSS declaration flex-shrink-0 did, applied to the Hire Us section's illustration image so it doesn't get squeezed by its flex siblings. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/components/ServicesSection.tsx`:
- Around line 41-47: Update the two new service-description paragraphs in
ServicesSection to use the text-body-md styling token instead of text-body-lg,
matching the intended design objective while preserving their existing spacing
and content.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 3e5f9bd4-a320-4f40-aca6-b370d89890c7
📒 Files selected for processing (8)
src/components/HireUsSection.tsxsrc/components/HomeHero.tsxsrc/components/MercenariesSection.tsxsrc/components/OurStorySection.tsxsrc/components/PortfolioSection.tsxsrc/components/ServicesSection.tsxsrc/components/TestimonialsSection.tsxsrc/lib/data/clients.ts
🚧 Files skipped from review as they are similar to previous changes (2)
- src/components/MercenariesSection.tsx
- src/components/PortfolioSection.tsx
| <p className="text-body-lg mb-3"> | ||
| Five service lines. One team assembled per project. | ||
| </p> | ||
| <p className="text-body-lg mb-3"> | ||
| Epic skills wielded by Web3 warriors.<br />Precision tools for | ||
| decentralized dominance. Mastery unmatched. | ||
| </p> |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Verify the text styling class against PR objectives.
The commit messages indicate that these two new service sentences should use the text-body-md token. However, they are currently using text-body-lg. Please verify if this deviation is intentional, or update the class to match the intended design.
🛠️ Proposed fix to match the PR objective
- <p className="text-body-lg mb-3">
+ <p className="text-body-md mb-3">
Five service lines. One team assembled per project.
</p>
- <p className="text-body-lg mb-3">
+ <p className="text-body-md mb-3">
Epic skills wielded by Web3 warriors.<br />Precision tools for
decentralized dominance. Mastery unmatched.
</p>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <p className="text-body-lg mb-3"> | |
| Five service lines. One team assembled per project. | |
| </p> | |
| <p className="text-body-lg mb-3"> | |
| Epic skills wielded by Web3 warriors.<br />Precision tools for | |
| decentralized dominance. Mastery unmatched. | |
| </p> | |
| <p className="text-body-md mb-3"> | |
| Five service lines. One team assembled per project. | |
| </p> | |
| <p className="text-body-md mb-3"> | |
| Epic skills wielded by Web3 warriors.<br />Precision tools for | |
| decentralized dominance. Mastery unmatched. | |
| </p> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/components/ServicesSection.tsx` around lines 41 - 47, Update the two new
service-description paragraphs in ServicesSection to use the text-body-md
styling token instead of text-body-lg, matching the intended design objective
while preserving their existing spacing and content.
Summary
No copy, voice, or structural changes beyond these four insertions. Join page untouched — out of scope per this pass.
Two items from the original spec were skipped as redundant on inspection, noted here rather than applied silently:
Test plan
bun run build— clean, no new errors or type issuesSummary by CodeRabbit
Content Updates
UI/UX Updates