Skip to content

Center the mobile menu icon in its button - #412

Merged
jasikpark merged 3 commits into
mainfrom
b-mobile-menu-logo-gap
Nov 12, 2026
Merged

jasikpark merged 3 commits into
mainfrom
b-mobile-menu-logo-gap

Conversation

@jasikpark

@jasikpark jasikpark commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

The mobile menu button is 2.75rem but its 16px icon sat in the top-left corner (8px / 20px insets), visible whenever the button's hover/open background shows. The button now centres its icon.

Before:

image

Fixed:

image

🤖 Generated with Claude Code

@github-actions

github-actions Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for docs-nebula ready!

Name Link
🔨 Latest commit 8c00913
🔍 Latest deploy log https://github.com/DefinedNet/nebula-docs/actions/runs/43292030397
😎 Deploy Preview Url https://7c4e57ce.docs-nebula.pages.dev
😎 Deploy Preview Alias https://b-mobile-menu-logo-gap.docs-nebula.pages.dev

@jasikpark
jasikpark force-pushed the b-mobile-menu-logo-gap branch from 2f50fc9 to 8373a4d Compare November 12, 2026 19:11
@jasikpark jasikpark changed the title Give the mobile menu button room before the logo Center the mobile menu icon and give the button room before the logo Nov 12, 2026
@jasikpark
jasikpark force-pushed the b-mobile-menu-logo-gap branch from 8373a4d to 0028443 Compare November 12, 2026 19:13
@jasikpark jasikpark changed the title Center the mobile menu icon and give the button room before the logo Center the mobile menu icon in its button Nov 12, 2026
@jasikpark
jasikpark marked this pull request as ready for review November 12, 2026 19:29
@jasikpark
jasikpark requested a review from bgolat November 12, 2026 19:30
@jasikpark
jasikpark enabled auto-merge (squash) November 12, 2026 19:37
@jasikpark
jasikpark merged commit 1fe8908 into main Nov 12, 2026
6 checks passed
@jasikpark
jasikpark deleted the b-mobile-menu-logo-gap branch November 12, 2026 19:44
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