Skip to content

www: docs: fix: Header nav items shouldn't wrap on mobile#2821

Closed
davis9001 wants to merge 14 commits intodenoland:mainfrom
davis9001:davis9001/mobile_move_social_icons
Closed

www: docs: fix: Header nav items shouldn't wrap on mobile#2821
davis9001 wants to merge 14 commits intodenoland:mainfrom
davis9001:davis9001/mobile_move_social_icons

Conversation

@davis9001
Copy link
Contributor

@davis9001 davis9001 commented Feb 5, 2025

Following up on #2820

I noticed the addition of the extra icon in the nav bar made the nav items in the Header newline wrap on small screens (mobile smaller than sm).
image

This pull request fixes that by doing the following:

  • On mobile: Hides GitHub social icon in Header

Some other changes I snuck in:

  • Reintroduced semi-transparent backgorund on header
  • Adjustments to spacing/alignment on Header items
  • Refactor ThemeToggle Island to be less Deno Fresh specific (more reusable)
  • Refactor social icons into SocialIcons component (removed duplication)
  • Added Discord social icon to Footer on all screen sizes

@davis9001 davis9001 closed this Nov 14, 2025
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