Skip to content

fix(marketing): dedupe nav icons, point Discord at # - #8

Merged
reecelikesramen merged 1 commit into
mainfrom
fix/nav-dup-discord
Sep 24, 2026
Merged

reecelikesramen merged 1 commit into
mainfrom
fix/nav-dup-discord

Conversation

@reecelikesramen

Copy link
Copy Markdown
Contributor

Prod currently renders github/discord/discussions/theme-toggle twice in the desktop nav — #7 added a stray <slot name="secondary" /> to the primary nav ul on top of the existing secondary nav.

Also points the header Discord icon at # (matching the footer and the docs site): discord.gg/pywire is not a valid invite code.

Verified locally: built site has one visible icon set, zero discord.gg refs, pnpm build + pnpm check pass.

- drop stray <slot name="secondary" /> from the primary nav ul (#7 added
  it, so github/discord/discussions/theme rendered in both desktop navs)
- discord icon links to # like the footer and docs do; discord.gg/pywire
  is not a valid invite
@reecelikesramen
reecelikesramen merged commit 18c8c4a into main Sep 24, 2026
2 checks passed

This branch was successfully deployed

1 active deployment
preview — d03de637 Deployed Sep 24, 2026 by github-actions[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