Skip to content

feat(home): port hero take C, centered and quiet - #437

Merged
harlan-zw merged 1 commit into
mainfrom
feat/home-hero-c
Oct 5, 2026
Merged

harlan-zw merged 1 commit into
mainfrom
feat/home-hero-c

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

Related to #435

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

Harlan picked take C, "Centered and quiet", from the hero lab. This PR puts it on the home page in place of the split hero: the trending rail, the agent chips, the intro paragraph and the promo chip all go. The four tiles and every band below them stay as they are.

  • The hero stacks the H1 with its rose dot, the CLI verb line, one caption, one search field and one mono claims line. The Braille names texture sits at the sides in stone, at low strength.
  • The verbs link to /cli#search, /cli#run, /cli#install and /cli#update. "The skilld CLI →" links to /cli. feat(cli): add the /cli landing page and a CLI header link #435 adds that page, so it should merge first.
  • The caption says 19 agents. The CLI defines AGENT_TARGETS: [AgentTarget; 19] in crates/skilld-core/src/target.rs, both in v3.6.0 and on main. The site's app/utils/agents.ts lists 12, so the caption does not use it.
  • HomeSearch gets the calm field from the lab. The solid button is gone. A / hint sits in the field, and it becomes an Enter key control once you type. Results open over the content below. app.vue already hides the header search on /, so only one panel opens. When the query is a GitHub URL, the submit control is labelled "Index repository".
  • The rail's selection code and fallback data are deleted (app/utils/homepage-person-skills.ts, app/data/homepage-person-skills.ts), with their CSS and the unused .home-display rules. SkillSourceList and AGENT_LOGOS stay, because the Skill page still uses them.
  • COPY.md records the verb line, the CLI link and the caption, and drops the old intro row. DESIGN.md replaces the split hero rule with the centered one.
  • The page <title>, the meta description and the "Hyped agent skills" text do not change. The SEO experiment and the production smoke check see the same page head.

Wording note: "finds and tracks skills" uses "track" as a verb, and GLOSSARY.md defines a track as a page of Skills. Harlan approved the caption wording, so it ships as written.

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

Harlan picked take C from the hero lab. The split hero with its proof
rail, agent chips, intro and promo chip gives way to a centered H1, the
CLI verb line, one caption, a calm search field and one claims line.

The caption's 19 agents is AGENT_TARGETS in the CLI's
crates/skilld-core/src/target.rs. The rail's selection code, fallback
data and CSS go with it. COPY.md records the new strings.
@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 5, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

Agent check in a headless browser against a local dev server with production feed data, since CI cannot cover it:

  • / at 1440 and 390, light and dark: no horizontal scroll, and the old rail and agent chips are gone.
  • The verb links resolve to /cli#search, /cli#run, /cli#install, /cli#update, and /cli.
  • The <title> and meta description match main. The H1 reads "Hyped agent skills, no bloat."
  • The / key focuses #home-skill-search. Typing opens one listbox over the tiles, and the / hint becomes the Enter control. Escape closes it, and Enter goes to /skills?q=….
  • The header search does not render on /.
  • The "Teach your agent skilld" popover shows npx skilld install skilld --global.
  • The browser console logged no errors. The dev HTML validator reports no new findings; the remaining table findings come from the existing weekly email preview.
1440 light 1440 dark
1440 light 1440 dark
390 light 390 dark
390 light 390 dark
Search open Promo popover
Search open Promo popover

@harlan-github-agent harlan-github-agent Bot added harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. and removed harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. labels Oct 5, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

🤖 MERGED

Harlan Agent Kit posted this automated review. It is not Harlan's personal review or approval. AI open source policy. Last updated: 2026-10-05 17:02 UTC.

GitHub merged this pull request.

  • Logged (40/100): The PR's six screenshots are inaccessible to this review, so the visual claims (no overlap, texture clear of copy, one panel) are unverified. View code
    • Ask an agent to verify and fix this finding
  • Logged (10/100): The hero texture loses its clear-copy mask on browser engines older than Chrome 120/Safari 15.4; texture shows behind the headline at 0.55 opacity. View code
    • Ask an agent to verify and fix this finding

Review completed. No finding above 80/100.

Selected findings run after merge and open separate pull requests.

@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. and removed harlan-agent-running An Agent holds a Task on this issue or pull request right now. labels Oct 5, 2026
@harlan-zw
harlan-zw merged commit 2440393 into main Oct 5, 2026
7 checks passed
@harlan-github-agent harlan-github-agent Bot added the harlan-agent-running An Agent holds a Task on this issue or pull request right now. label Oct 5, 2026
@harlan-github-agent harlan-github-agent Bot removed harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Oct 5, 2026
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