Skip to content

feat(home): trim the trending band and quiet the watch CTA - #459

Merged
harlan-zw merged 1 commit into
mainfrom
feat/home-trending-cta
Oct 6, 2026
Merged

harlan-zw merged 1 commit into
mainfrom
feat/home-trending-cta

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

❓ Type of change

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

📚 Description

The homepage trending band showed a run command on single-Skill cards only, so one card looked different from the rest. Its header also carried a repository count and Week, Month, and All time tabs that pulled attention from the cards.

  • Trending cards drop the run chip. The repository count and range tabs are gone.
  • One "All trending skills" link to /skills/trending sits under the grid, styled like "All tracks".
  • The trending and tracks leads have new copy.
  • "Keep up with skill changes." fits on one line.
  • "Watch your starred repos" is a neutral outline button, and the example digest is a text link, so the band has no red fill.

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

The trending cards showed a run command on single-Skill repositories only, which read as inconsistent. The band header carried a repository count and range tabs that competed with the cards. One link to the full trending board replaces them, matching the tracks band.
@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 6, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

Agent check, since CI cannot cover it: a local dev build with homepage feeds proxied from production, viewed headless at 1280px and 390px.

  • The trending band shows six cards, no run chip, no count, no range tabs, and an "All trending skills" link under the grid.
  • "Keep up with skill changes." renders as one line at 1280px (44px tall) and at 390px.
  • The page has no horizontal scroll at 390px.

@harlan-github-agent harlan-github-agent Bot added harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. harlan-agent-running An Agent holds a Task on this issue or pull request right now. and removed harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. labels Oct 6, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 6, 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-06 13:12 UTC.

GitHub merged this pull request.

  • Logged (20/100): The trending section's only outbound link sends 'this week' readers to the month board. 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. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. and 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 6, 2026
@harlan-zw
harlan-zw merged commit b5fda11 into main Oct 6, 2026
7 checks passed
@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 6, 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