Skip to content

docs(readme): add Claude Code CTA above the fold (V5 d)#128

Closed
naorsabag wants to merge 1 commit into
docs/readme-v3-badge-colorsfrom
docs/readme-v5-claude-code-cta
Closed

docs(readme): add Claude Code CTA above the fold (V5 d)#128
naorsabag wants to merge 1 commit into
docs/readme-v3-badge-colorsfrom
docs/readme-v5-claude-code-cta

Conversation

@naorsabag
Copy link
Copy Markdown
Owner

Summary

Closes V5 killer-line (d) "Claude Code skill section above the fold" from openhop-launch/03-readme-and-landing.md.

V5 audit flagged that the Claude Code Install instructions sit ~3 viewports down on a 15" laptop. The Claude Code platform badge is above the fold, but a passive badge isn't a "section." Adds a one-line CTA between the hero GIF and the nav, anchored to #install-options:

<p align="center">
  <a href="#install-options"><b>→ Add to Claude Code in one command</b></a>
</p>

So a Claude Code user can jump from "saw the GIF, want it" → install instructions in one click without scrolling.

Not in this PR — V5 (c) is structurally blocked

V5 (c) "Try it live link before any CLI command" can't be cleanly satisfied with the current hero shape:

  • The hero command is npx openhop init (per the V1 revert — install-first is intentional)
  • There's no hosted playground at v0.1 (see 02-hosted-playground.md)
  • A "Try it live" link with nothing to point at would be worse than no link

Will close on its own once 02-hosted-playground.md ships.

Base branch note

Targets docs/readme-v3-badge-colors (PR #127) → stacks under V2 (#126) → V1 (#125) → master. Cascade auto-retargets on each merge.

Test plan

  • Verify the CTA scrolls to the Install section on click.
  • On a 15" laptop, the CTA is visible above the fold (it's between the GIF and the nav, so it sits ~80% down the hero — adds zero scroll).
  • Doesn't break the hero rhythm: tagline → vs-callout → privacy → standard badges → platform badges → GIF → CTA → nav.
  • CI green.

🤖 Generated with Claude Code

V5 audit from openhop-launch/03-readme-and-landing.md flagged that the
Claude Code skill section sits ~3 viewports down on a 15" laptop. The
Claude Code platform badge is above the fold, but a passive badge
isn't a section. Adds a one-line CTA between the GIF and the nav
anchored to #install-options, so a Claude Code user can jump from
"saw the GIF, want it" to install instructions in one click without
scrolling.

V5 c ("Try it live link before any CLI command") is intentionally not
in this PR - it's structurally blocked until the hosted playground
ships (openhop-launch/02-hosted-playground.md), since npx openhop init
is the chosen hero command and there's no playground link to point at.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 11, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (1)
  • master

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: a001c3f9-49fe-4642-926f-b7d99105950c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/readme-v5-claude-code-cta

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

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

@naorsabag naorsabag closed this May 11, 2026
@naorsabag naorsabag deleted the docs/readme-v5-claude-code-cta branch May 15, 2026 16:00
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