Skip to content

fix(docs): enlarge the hero logo on tablet/mobile#528

Merged
btravers merged 2 commits into
mainfrom
fix/hero-logo-mobile
Jul 3, 2026
Merged

fix(docs): enlarge the hero logo on tablet/mobile#528
btravers merged 2 commits into
mainfrom
fix/hero-logo-mobile

Conversation

@btravers

@btravers btravers commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

The shared theme only enlarges the hero mark at >=960px, so on tablet and mobile it rendered at its small intrinsic size (~64px). This sizes it to ~208px there too. Verified locally at 768 and 390.

🤖 Generated with Claude Code

The shared theme only enlarges the hero mark at >=960px, so below that it
rendered at its small intrinsic size (~64px). Size it to ~208px for
tablet/mobile too.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 3, 2026 14:11

Copilot AI left a comment

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.

Pull request overview

Adjusts the VitePress docs theme CSS so the home-page hero logo remains legible on tablet/mobile viewports (<960px), matching the intended larger sizing currently applied only on desktop.

Changes:

  • Adds a <960px media query that sets explicit sizing for .VPHero .image-src on smaller screens.
  • Applies object-fit: contain to keep the hero mark from distorting when resized.

Comment thread docs/.vitepress/theme/custom.css Outdated
Use min(208px, 62vw) for both width and height so the mark shrinks
proportionally on very narrow screens instead of going non-square.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@btravers

btravers commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Good catch — fixed. Replaced width: 208px; height: 208px; max-width: 62vw with width: min(208px, 62vw); height: min(208px, 62vw) so both dimensions shrink together and the mark stays square at any width.

@btravers btravers merged commit 889e5fa into main Jul 3, 2026
9 checks passed
@btravers btravers deleted the fix/hero-logo-mobile branch July 3, 2026 14:24
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.

2 participants