Skip to content

feat(quickstart): PR2 — guided vertical stepper with output hints#27

Closed
BunsDev wants to merge 1 commit into
feat/section-overhaul-foundationfrom
feat/section-overhaul-quickstart
Closed

feat(quickstart): PR2 — guided vertical stepper with output hints#27
BunsDev wants to merge 1 commit into
feat/section-overhaul-foundationfrom
feat/section-overhaul-quickstart

Conversation

@BunsDev

@BunsDev BunsDev commented Jul 7, 2026

Copy link
Copy Markdown
Member

PR2 of the section overhaul — Quick Start

Stacked on #26 (foundation). Review/merge that first.

Rebuilds Quick Start from three flat parallel cards into a semantic <ol>
stepper that reads as a guided sequence, on the hero's continuity-rail motif.

What changed

  • Sequence, not a grid: numbered nodes threaded on a gradient connector; the final node lights up (is-now + now-pulse) as "you're running now".
  • Each step = a glass card with a syntax-accented command (static token spans; the clean plaintext stays in data-copy), an expected-output "expect" hint (what success looks like), and a short note.
  • Accessibility: copy now announces "Copied: …" via an sr-only aria-live region and restores the step-specific button label (was hardcoded).
  • Copy tweak: step 3 retargeted to Forge (coven run forge "pick up where we left off") to tie into the hero narrative; migrated off the legacy #8E3DFF violet.
  • Mobile: vertical at every breakpoint — no 3→1 collapse to manage.

Verification

  • npm run build ✓ · npm run check ✓ (required copy strings intact) · no console errors on load (CDP).
  • Desktop + 390px mobile headless captures: stepper, connector, colored tokens, expect hints, and the "now" node all render correctly.

🤖 Generated with Claude Code

Rebuild Quick Start from three flat parallel cards into a semantic <ol>
stepper that reads as a sequence, on the hero's continuity-rail motif.

- Numbered nodes threaded on a gradient connector; the final node lights
  up (is-now + now-pulse) as "you're running now".
- Each step is a glass card with a syntax-accented command (static token
  spans; clean plaintext stays in data-copy), an expected-output "expect"
  hint (what success looks like), and a short note.
- Copy now announces success to screen readers via an sr-only aria-live
  region and restores the step-specific button label.
- Step 3 command retargeted to Forge ("pick up where we left off") to tie
  into the hero narrative; migrated to the new violet palette.
- Vertical at every breakpoint, so mobile just works (no 3->1 collapse).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 7, 2026 04:42
@vercel

vercel Bot commented Jul 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
coven Ready Ready Preview Jul 7, 2026 4:42am
coven-landing Ready Ready Preview Jul 7, 2026 4:42am

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR overhauls the Quick Start section from a 3-card grid into a semantic, guided vertical stepper that aligns with the site’s continuity-rail visual language, adding syntax-accented commands plus “expected output” hints and improved copy-to-clipboard accessibility.

Changes:

  • Replaces the Quick Start layout with an <ol> stepper UI (numbered rail, connector, “now” final step).
  • Adds new Quick Start styling for stepper layout, syntax-token coloring, and output “expect” hints.
  • Enhances copy-to-clipboard behavior to restore per-step button labels and announce copy success via an aria-live region.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/styles/global.css Adds stepper/rail/node styles, syntax token accents, and expected-output hint styling for Quick Start.
src/scripts/main.js Updates Quick Start copy-to-clipboard logic to restore per-step labels and announce copy events.
src/components/QuickStart.astro Rebuilds Quick Start markup into a semantic ordered stepper with tokenized command display + output hints and an sr-only live region.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/scripts/main.js
Comment on lines 221 to 225
btn.addEventListener('click', function () {
navigator.clipboard.writeText(btn.getAttribute('data-copy')).then(function () {
var cmd = btn.getAttribute('data-copy');
navigator.clipboard.writeText(cmd).then(function () {
btn.classList.add('is-copied');
btn.innerHTML = CHECK_SVG;
@BunsDev

BunsDev commented Jul 7, 2026

Copy link
Copy Markdown
Member Author

Superseded by #33 — a concurrent session had interleaved two unrelated commits (px→rem conversion, multi-screen support) into this stack. Re-cut cleanly in an isolated worktree with only the six signed overhaul commits. Closing in favour of #33.

@BunsDev BunsDev closed this Jul 7, 2026
@BunsDev
BunsDev deleted the feat/section-overhaul-quickstart branch July 7, 2026 11:20
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.

2 participants