Skip to content

Fix landing page install UX and hero alignment - #16

Merged
samlehoy merged 1 commit into
masterfrom
feat/landing-page-polish
Aug 17, 2026
Merged

samlehoy merged 1 commit into
masterfrom
feat/landing-page-polish

Conversation

@samlehoy

Copy link
Copy Markdown
Owner

Summary

  • Removed the redundant install pill from the nav (desktop + mobile) — it duplicated the same CTA already in the hero/install sections.
  • Added a collapsible beginner install walkthrough at #install (install Node.js → open terminal → paste command → verify with plx --version), a plain-language gloss for the Deezer ARL cookie, and pointed the "grab yours" link at the actual docs/CONFIG.md#getting-the-cookies section instead of the same generic docs folder as the footer "Docs" link.
  • Unified the two differently-behaving install-command pills (one silently left the site, one didn't) into a single CopyInstallPill component that copies the command to clipboard, with a Clipboard-API-unavailable fallback and an accessible copied-state announcement.
  • Fixed the hero content column rendering 200–330px off the shared left rail every other section uses at desktop widths (mx-auto combined with lg:w-[54%] was centering it independently in the viewport instead of anchoring it to the same max-w-[1440px] mx-auto container as the rest of the page).
  • Minor: footer Github link now matches its nav twin's target="_blank" rel="noreferrer".

Driven by two /impeccable critique passes this session (Design Health Score 26/36 → 19/28 across runs, focused on the install/onboarding flow and then the hero-alignment regression); findings and evidence are in web/.impeccable/critique/.

Test plan

  • npm run build (tsc + vite) clean
  • Design detector (detect.mjs) clean, zero findings
  • Hero/nav/install left-edge alignment re-measured at 1024/1440/1920px — 0px offset (previously up to 331px)
  • Copy-to-clipboard verified via mouse click and keyboard (Tab + Enter) at desktop and mobile viewports
  • Mobile nav, tier picker, and install walkthrough interaction pass — zero console errors
  • Manual look in a real browser before merge

🤖 Generated with Claude Code

- Remove redundant install pill from nav (desktop + mobile)
- Add beginner install walkthrough, ARL cookie gloss, and fix
  duplicate Docs/grab-yours link
- Unify install command into one copy-to-clipboard control
- Fix hero content column misaligned with rest of page at desktop
  widths (mx-auto + lg:w-[54%] was centering it independently
  instead of anchoring to the shared max-w-[1440px] rail)

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@samlehoy
samlehoy merged commit 439968d into master Aug 17, 2026
5 of 7 checks passed
@samlehoy
samlehoy deleted the feat/landing-page-polish branch August 17, 2026 23:32
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