Skip to content

feat: adapt homepage avatar size to intro column height#94

Merged
kaiiiichen merged 1 commit into
mainfrom
feat/adaptive-identity-avatar
Jul 6, 2026
Merged

feat: adapt homepage avatar size to intro column height#94
kaiiiichen merged 1 commit into
mainfrom
feat/adaptive-identity-avatar

Conversation

@kaiiiichen

Copy link
Copy Markdown
Owner

Summary

  • Add IdentityPhotoColumn with a ResizeObserver that sizes the homepage portrait on desktop from the intro column height minus social-link height and gap
  • Keep greeting copy and social icons in place; only the avatar shrinks when the right column is short so icons are not pressed against the photo bottom
  • Remove md:h-full from HomeSocialLinks so icons use their natural height instead of stretching into leftover grid space

Test plan

  • Open / on desktop (≥768px) and confirm avatar + social links spacing looks balanced with current intro copy
  • Temporarily shorten the Greeting paragraphs and confirm the avatar shrinks while social icons keep a consistent gap below it
  • Resize the browser window and confirm avatar size updates smoothly
  • Check mobile layout still uses the fixed square avatar (aspect-square)
  • Run npm run lint && npm run typecheck && npm run build

Made with Cursor

Shrink the identity photo on desktop when greeting copy is short so
social links keep breathing room below the portrait instead of sitting
flush against a fixed square crop.

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Jul 6, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
kaichen.dev Ready Ready Preview, Comment Jul 6, 2026 3:18am

@kaiiiichen kaiiiichen merged commit 81a0049 into main Jul 6, 2026
4 checks passed
@kaiiiichen kaiiiichen self-assigned this Jul 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