Skip to content

fix(header): use xl breakpoint and animate hamburger icon#425

Open
vitormattos wants to merge 3 commits into
mainfrom
fix/header-menu-dynamic-breakpoint
Open

fix(header): use xl breakpoint and animate hamburger icon#425
vitormattos wants to merge 3 commits into
mainfrom
fix/header-menu-dynamic-breakpoint

Conversation

@vitormattos

@vitormattos vitormattos commented Jul 6, 2026

Copy link
Copy Markdown
Member

Summary

  • lower the navbar collapse breakpoint to xl so the menu switches before the header wraps
  • keep the responsive menu stable for multilingual content by relying on Bootstrap/CSS only
  • animate the hamburger toggler into an X when the menu is open

Validation

  • built the site assets in the PHP container with npm run build-assets
  • verified the menu behavior in the browser at multiple widths
  • confirmed the toggler morphs into an X via aria-expanded/CSS state
image image

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://libresign.github.io/site-preview/pr-preview/pr-425/

Built to branch main at 2026-07-06 20:01 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos vitormattos force-pushed the fix/header-menu-dynamic-breakpoint branch from 056a256 to 1911b77 Compare July 6, 2026 19:58
@vitormattos vitormattos requested review from YvesCesar and henmohr July 6, 2026 20:07
@henmohr

henmohr commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

When first acessing the site its showing like the menu has been clicked, but it wasn't.

image

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