Repository navigation
Refine Flows headline typography and workflow copy - #70
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe Flows page updates messaging copy and accessibility text, broadens the messaging title type to ChangesFlows heading update
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The Flows copy, heading contract, and typography changes are localized with no established user-facing regression, so the PR is ready to merge. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches 💡 1📝 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. A rabbit hops where new words flow Comment |
|
Preview deployed!
This is a Cloudflare Workers preview version of this PR's build. |
Adjust the Flows hero headline to Sora weight 500, a 4.2rem maximum size, 1.05 line height, and -0.03em letter spacing. Existing mobile sizing remains intact.
Replace the workflow heading with “Agents can ignore instructions.” followed by an explicit line break and “Flows can’t.” Update its accessible label and allow MessagingFeature titles to accept React content.
Validation: TypeScript passed (
tsc --noEmit --incremental false -p web/tsconfig.json);git diff --checkpassed. Based on current main after the merge of #68.Note
Low Risk
Marketing copy, CSS typography, and a shared component prop type only—no auth, data, or backend changes.
Overview
Updates the Flows landing page hero and workflow messaging, plus a small layout fix for sticky navigation.
The hero h1 is tuned to weight 500, a slightly larger max size (4.2rem), 1.05 line height, and -0.03em letter spacing. Horizontal overflow on
.pageswitches fromhiddentoclipso decorative overflow is clipped without turning the page into a scroll container that breaks sticky nav behavior.The workflow block replaces “Skills make suggestions. Flows enforce steps.” with “Agents can ignore instructions.” / “Flows can’t.”, with matching
aria-labeland aMessagingFeaturetitle that can be React content (line break). SiteNav on/flowsno longer passeshideLinks, so standard nav links show again on this page.Reviewed by Cursor Bugbot for commit 89cbab5. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by cubic
Refines the Flows hero headline typography, workflow copy, and navigation.
Typography & layout
overflow-xis nowclipso decorative overflow doesn't create a scroll container that breaks sticky positioning.Workflow copy
MessagingFeaturetitles now accept React content instead of plain strings.Written for commit 89cbab5. Summary will update on new commits.