Skip to content

Collapse hamburger menue#53

Merged
shrinkray merged 2 commits into
mainfrom
feat/content
May 28, 2026
Merged

Collapse hamburger menue#53
shrinkray merged 2 commits into
mainfrom
feat/content

Conversation

@shrinkray
Copy link
Copy Markdown
Owner

  • Updated src/components/Header.astro to remove flex from the nav { ... } CSS rule.
  • Updated the toggle script to:

open: remove hidden and add flex
close: add hidden and remove flex

shrinkray added 2 commits May 27, 2026 21:56
- remove flex from the nav { ... } CSS rule.
- update toggle script to:
- open: remove hidden and add flex
- close: add hidden and remove flex
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the header’s mobile navigation behavior so the hamburger menu properly collapses/expands by using Tailwind’s hidden/flex utilities instead of the project’s display-none helper class.

Changes:

  • Switch header nav default mobile state to hidden (while keeping sm:flex for desktop).
  • Update the hamburger toggle script to add/remove hidden and flex appropriately, and fix the #second-line selector typo.
  • Bump caniuse-lite in package-lock.json.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
src/components/Header.astro Aligns mobile nav visibility and toggle logic with Tailwind hidden/flex, fixing menu collapse behavior.
package-lock.json Lockfile refresh updating caniuse-lite.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@shrinkray shrinkray merged commit 168b718 into main May 28, 2026
2 checks passed
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