Skip to content

feat(website): adopt new hunk logo, flat favicon, and brand guide - #1145

Merged
benvinegar merged 2 commits into
mainfrom
claude/hunk-logo-redesign-d1047c
Oct 5, 2026
Merged

benvinegar merged 2 commits into
mainfrom
claude/hunk-logo-redesign-d1047c

Conversation

@benvinegar

Copy link
Copy Markdown
Member

Summary

  • Header now shows the flat "h" mark beside the lowercase `hunk` wordmark (24px, `currentColor`).
  • Favicon is the flat mark as a single seamless path (no hairline); `icon.png` and `apple-icon.png` regenerated from it, and `favicon.svg` is linked first in `PageHead`.
  • Adds clean logo files in `website/public/brand/` (flat mark, 3D mark, wordmark, lockup) and `docs/brand.md` covering logo versions and usage, palette, and JetBrains Mono.

Notes

  • Colors are unchanged. The favicon green (`#65d291`) still differs from the site accent (`#1f7a35`); the brand guide flags it.
  • Not included: `og.png` and `website/public/docs/favicon.svg` are still on the old branding.
  • Empty changeset (website-only).

Verification

  • `astro check`: 0 errors, warnings, hints.
  • Rendered the home, docs, extensions pages and a 390px mobile view locally with the new header.

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
hunk-web Ready Ready Preview Oct 5, 2026 12:36pm UTC

Request Review

@greptile-apps

greptile-apps Bot commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 4/5

[Low risk] Updates website branding assets and styling.

The PR should not merge until the wider logo fits alongside navigation at narrow marketing viewport widths.

Findings

  1. P1 Narrow header clips links ▶
Fix with agent prompt
### Issue 1
website/src/styles/brand.css:60-62
The new 24px mark and 9px gap make the logo wider, but it still cannot shrink and the marketing navigation still uses breakpoints set for the old text-only logo. At narrow widths just above 430px or 344px, the links no longer fit. Because horizontal overflow is hidden, right-side navigation can be clipped instead of remaining accessible. Adjust the layout or breakpoints for the wider logo.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

The PR adds a flat header mark, updates the marketing favicon and icon assets, and publishes standalone logo files with a brand guide.

  • The wider header logo needs a responsive-layout adjustment at narrow marketing viewport widths.

Reviews (1) · Last reviewed commit: "feat(website): adopt new hunk logo, flat..."

Comment on lines +60 to +62
display: inline-flex;
align-items: center;
gap: 9px;

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.

P1 Narrow header clips links

The new 24px mark and 9px gap make the logo wider, but it still cannot shrink and the marketing navigation still uses breakpoints set for the old text-only logo. At narrow widths just above 430px or 344px, the links no longer fit. Because horizontal overflow is hidden, right-side navigation can be clipped instead of remaining accessible. Adjust the layout or breakpoints for the wider logo.

Prompt To Fix With AI
This is a comment left during a code review.
Path: website/src/styles/brand.css
Line: 60-62

Comment:
**Narrow header clips links**

The new 24px mark and 9px gap make the logo wider, but it still cannot shrink and the marketing navigation still uses breakpoints set for the old text-only logo. At narrow widths just above 430px or 344px, the links no longer fit. Because horizontal overflow is hidden, right-side navigation can be clipped instead of remaining accessible. Adjust the layout or breakpoints for the wider logo.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@benvinegar
benvinegar merged commit a3321c8 into main Oct 5, 2026
15 checks passed

This branch was successfully deployed

1 active deployment
Preview — 0bc5d86c Deployed Oct 5, 2026 by vercel[bot]
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