Repository navigation
docs(site): a shared lesson introduces devcoach to people who don't have it - #130
Closed
UltimaPhoenix wants to merge 1 commit into
Closed
UltimaPhoenix wants to merge 1 commit into
UltimaPhoenix wants to merge 1 commit into
Conversation
…ave it Every share link lands someone on the docs site, and often that someone has never heard of devcoach. The page spoke only to existing users: its first words after the lesson were "No dashboard answered on 127.0.0.1:7860. Start it with devcoach ui…", and the way in for a newcomer was one small "Install it" link at the bottom of the panel. - the eyebrow names the sender: "Alex shared a devcoach lesson with you" (unchanged wording when the share is anonymous) - when the probe finds no dashboard (down, blocked, or a browser that can't tell), a "New to devcoach?" panel comes first: one sentence on what devcoach does, Install devcoach, See how it works, and a note that the link keeps the lesson for later. A running dashboard hides it - the import panel's old install hint becomes a neutral "Or paste the code into your agent" - the sticky aside caps its height so its end stays reachable on short windows Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NrnCoNszDPBcY6KggBT5JE
Contributor
|
Thank you for your contribution! Before we can merge it, please read our Contributor License Agreement and sign it by posting the following comment on this pull request: I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
5 of 7 tasks
Owner
Author
|
Superseded by #147 — same change, rebased onto develop and re-committed under the repository owner so the CLA check passes. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
Every share link lands someone on the docs site's
/lessonpage, and often that someone has never heard of devcoach. That makes shared lessons devcoach's referral channel. The page spoke only to existing users. Its first words next to the lesson were "No dashboard answered on 127.0.0.1:7860. Start it withdevcoach ui…", and the only way in for a newcomer was one small "Install it" link at the bottom of the panel.Now the page:
People who already run devcoach see the page exactly as before.
Part of a series of small website PRs from a product/marketing review. Each PR stands on its own. The buttons here are still indigo because the branch is cut from
develop; #125 turns them teal.Before / after
No dashboard detected (a newcomer, or a user whose dashboard isn't running), dark

No dashboard detected, light

Dashboard running. Unchanged apart from the sender's name and the neutral paste hint.

Mobile (390 px, whole page): the lesson first, then the pitch, then Import
Captures: production builds served under
/dev-coach/, a share code built from the screenshot fixture's Docker lesson (shared by "Alex"), and Playwright with the dashboard probe stubbed as up or down. Images live on theclaude/wizardly-hamilton-k6nc1c-pr-mediabranch, not in this diff.Changes
website/src/pages/lesson.tsx+lesson.module.css:down,blocked-permission,needs-permission, and Safari'sunsupported. It is hidden while checking and when the dashboard is up./install), See how it works (→ the docs home), and "Then come back: this link keeps the lesson, ready to import."100vhminus the navbar) with its own overflow, so its end stays reachable on short windows now that it can hold two panels.CHANGELOG.md: one entry under[Unreleased]. Needs a trivial merge if a sibling PR lands first.The probe, the import link, the decoder and every privacy property of the page are untouched.
Checklist
npm run lintpasses (Biome). The website is outside Biome's scope.npm run typecheck. Nosrc/changes.npm run typecheck --prefix websitealready fails ondevelop(TypeScript 7 rejects the inheritedbaseUrl); the production build compiles the page.npm run build --prefix websiteandnpm test --prefix website(8/8) passCHANGELOG.mdwhere relevantGenerated by Claude Code