Skip to content

Refine mobile hero spacing and investor logo rotation - #74

Merged
willwashburn merged 1 commit into
mainfrom
codex/mobile-investor-marquee
Sep 15, 2026
Merged

willwashburn merged 1 commit into
mainfrom
codex/mobile-investor-marquee

Conversation

@willwashburn

@willwashburn willwashburn commented Sep 15, 2026 •

Copy link
Copy Markdown
Member

Summary

Improve the mobile hero hierarchy on the home and Flows pages with larger headlines, smaller supporting text, and consistent spacing: 14px between headline and copy, 28px before buttons, 40px before investors, and 12px between the investor label and logos.

Show two investor logos at a time on mobile, fading between pairs every five seconds and starting with Y Combinator and Cortical Ventures. Preserve desktop logo order and use a static list for reduced-motion users. Tighten the investor label-to-logo gap on all sizes.

Validation

  • Local Playwright checks at 375px, 600px, 680px, and 1280px across the affected layouts.
  • Verified paired logo visibility, desktop visibility, reduced-motion fallback, and no horizontal overflow.
  • Measured the 14px / 28px / 40px / 12px spacing on both mobile heroes.
  • git diff --check passed.

Note

Low Risk
Presentation-only CSS and marketing component changes with no auth, data, or API impact.

Overview
Refines mobile hero hierarchy on the landing and Flows pages: larger headlines (clamp(2.75rem, 12vw, 4rem)), tighter supporting copy (0.95rem), and explicit spacing for copy blocks, CTAs, and the investor strip via --investor-strip-padding-top (40px on Flows, 2.5rem on the home hero section).

Changes InvestorStrip for small screens by showing two logos at a time with a CSS fade rotation (~5s per pair), while desktop keeps the full row. Logo markup is centralized in a logos array reused by both layouts; prefers-reduced-motion restores the static desktop row on mobile.

Reviewed by Cursor Bugbot for commit 5a09abc. Bugbot is set up for automated code reviews on this repo. Configure here.


Summary by cubic

Refines the home and Flows mobile heroes with larger headlines, more compact supporting text, and consistent spacing: 14px between headline and copy, 28px before buttons, 40px before investors, and 12px between the investor label and logos. On screens up to 680px, the investor strip now shows two logos at a time, starting with Y Combinator and Cortical Ventures and fading to the next pair every five seconds.

Details

  • Desktop investor order remains unchanged.
  • Reduced-motion users see the full investor list without animation.

Written for commit 5a09abc. Summary will update on new commits.

Review in cubic

@coderabbitai

coderabbitai Bot commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: bc04d10c-f13f-4754-bdfa-5e01c65c3351

📥 Commits

Reviewing files that changed from the base of the PR and between 7195bd0 and 5a09abc.

📒 Files selected for processing (4)
  • web/app/flows/flows.module.css
  • web/app/landing.module.css
  • web/components/InvestorStrip.tsx
  • web/components/investor-strip.module.css

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

Mobile hero layouts now use larger headlines, explicit lead text sizing, adjusted spacing, and revised investor strip padding. The investor strip now supports responsive logo pairs with crossfade animation and a reduced-motion fallback.

Changes

Mobile presentation

Layer / File(s) Summary
Flows mobile hero styles
web/app/flows/flows.module.css
The mobile flows hero now uses updated padding, gaps, headline sizing, and lead text typography.
Landing mobile hero styles
web/app/landing.module.css
The mobile landing hero now uses updated padding, headline sizing, subtitle typography, and CTA spacing.
Responsive investor strip rendering
web/components/InvestorStrip.tsx, web/components/investor-strip.module.css
Investor logos are defined as reusable data, rendered in desktop and mobile variants, and crossfaded on mobile. Reduced-motion users see the desktop layout without animation.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Suggested reviewers: khaliqgant

Merge Risk: ⚪ Minimal · up to 5a09a

The mobile investor rotation starts with the required logos and crossfades correctly; no actionable merge risk remains.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning 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 1 functions across 1 files. (3 skipped: 3 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main changes: mobile hero spacing refinements and investor logo rotation.
Description check ✅ Passed The description directly explains the mobile hero updates, investor logo rotation, desktop behavior, reduced-motion fallback, and validation performed.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

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 1 functions across 1 files. (3 skipped: 3 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/mobile-investor-marquee

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.

❤️ Share

A rabbit reviews the logos in line
Two hop together, then fade by design
The headlines grow tall
The spacing fits all
Reduced motion keeps the rhythm kind

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 15, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-15T04:46:07.307430Z 5a09abc PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions

Copy link
Copy Markdown
Contributor

Preview deployed!

Environment URL
Web https://772e4b96-agentrelay-web.agent-workforce.workers.dev

This is a Cloudflare Workers preview version of this PR's build.

@willwashburn
willwashburn merged commit 3a0c52d into main Sep 15, 2026
4 checks passed
@willwashburn
willwashburn deleted the codex/mobile-investor-marquee branch September 15, 2026 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant