Skip to content

Conversation

@ch0rizo
Copy link
Contributor

@ch0rizo ch0rizo commented Jan 14, 2026

This resolves an issue where the mobile menu toggle becomes hard to see when the header is partially scrolled. Because the header is not sticky, the toggle would remain in its old position and appear almost off-screen when opening the menu.

The change ensures the toggle aligns with the fixed search field when the menu is open. This improves visibility and overall usability.

REF: Fix menu toggle alignment on scroll #5799

Result

If you've made visual changes, please check the boxes below and include images showing the changes. Descriptions are appreciated.

  • Changes look good on both light and dark theme.
  • Changes look good with different viewports (mobile, tablet, etc.).
  • Changes look good with slower Internet connections.

Testing

  • I have thoroughly tested my changes.

This resolves an issue where the mobile menu toggle becomes hard to see when the header is partially scrolled. Because the header is not sticky, the toggle would remain in its old position and appear almost off-screen when opening the menu.

The change ensures the toggle aligns with the fixed search field when the menu is open. This improves visibility and overall usability.
@ch0rizo ch0rizo requested review from a team and Viljen789 January 14, 2026 12:18
@ch0rizo ch0rizo self-assigned this Jan 14, 2026
@github-actions github-actions bot added the review-needed Pull requests that need review label Jan 14, 2026
Copy link
Contributor

@Viljen789 Viljen789 left a comment

Choose a reason for hiding this comment

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

Great changes by williamsaether !

@ch0rizo ch0rizo merged commit 011c971 into master Jan 14, 2026
7 checks passed
@ch0rizo ch0rizo deleted the fix-nav-close-location branch January 14, 2026 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-needed Pull requests that need review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants