Skip to content

Menu scroll fix#68

Merged
sampoyigi merged 6 commits into4.xfrom
menu-scroll-fix
Nov 24, 2025
Merged

Menu scroll fix#68
sampoyigi merged 6 commits into4.xfrom
menu-scroll-fix

Conversation

@obinnaelviso
Copy link
Copy Markdown
Contributor

This pull request updates the anchor link scrolling behavior to improve user experience when navigating to sections of a page with a sticky header. The change ensures that the scrolled-to section is not hidden behind the sticky header and prevents the default browser behavior for anchor links.

Improvements to anchor link scrolling:

  • Added event.preventDefault() to prevent the default anchor link behavior, ensuring smooth scrolling is handled by JavaScript.
  • Adjusted the scroll position calculation to subtract the height of the .sticky-top header, so the target section is fully visible below the sticky header.
  • Changed the scroll animation duration from 1000ms to 0ms for immediate scrolling.

@sampoyigi sampoyigi merged commit 50145f5 into 4.x Nov 24, 2025
4 checks passed
@sampoyigi sampoyigi deleted the menu-scroll-fix branch November 24, 2025 11:35
@sampoyigi sampoyigi mentioned this pull request Nov 24, 2025
sampoyigi added a commit that referenced this pull request Nov 24, 2025
### Added
- Improve category menu query to filter by location.
- **Local Header Component:** Add current page support to display opening schedule on reservation page.

### Fixed
- Add return type declaration for menus count query closure.
- **Booking Component:** Remove duplicate `data-control="booking"` element.
- **Local Header Component:** Fix menu scroll issue. (#68)
- Ensure current order type is updated during boot process. (#65)
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