menu item scroll problem in mobile fixed
This commit is contained in:
@@ -78,19 +78,15 @@ function Header({ showMenu = false }: HeaderProps) {
|
|||||||
const element = isOnMainPage ? document.getElementById(targetId) : null;
|
const element = isOnMainPage ? document.getElementById(targetId) : null;
|
||||||
|
|
||||||
if (element) {
|
if (element) {
|
||||||
// Safari-compatible scrolling with fallback
|
// Close drawer first, then scroll after animation completes
|
||||||
const headerOffset = headerHeight || 0;
|
if (isDrawerOpen) {
|
||||||
const elementPosition = element.getBoundingClientRect().top;
|
setIsDrawerOpen(false);
|
||||||
const offsetPosition = elementPosition + window.pageYOffset - headerOffset;
|
// Wait for drawer animation to complete (300ms) before scrolling
|
||||||
|
setTimeout(() => {
|
||||||
try {
|
performScroll(element);
|
||||||
window.scrollTo({
|
}, 350);
|
||||||
top: offsetPosition,
|
} else {
|
||||||
behavior: 'smooth'
|
performScroll(element);
|
||||||
});
|
|
||||||
} catch (e) {
|
|
||||||
// Fallback for older Safari versions
|
|
||||||
window.scrollTo(0, offsetPosition);
|
|
||||||
}
|
}
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
@@ -105,6 +101,31 @@ function Header({ showMenu = false }: HeaderProps) {
|
|||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
const performScroll = (element: HTMLElement) => {
|
||||||
|
// Safari and mobile-compatible scrolling with multiple fallbacks
|
||||||
|
const headerOffset = headerHeight || 0;
|
||||||
|
const elementPosition = element.getBoundingClientRect().top;
|
||||||
|
const offsetPosition = elementPosition + (window.pageYOffset || window.scrollY || document.documentElement.scrollTop) - headerOffset;
|
||||||
|
|
||||||
|
// Try smooth scrolling first
|
||||||
|
try {
|
||||||
|
window.scrollTo({
|
||||||
|
top: offsetPosition,
|
||||||
|
behavior: 'smooth'
|
||||||
|
});
|
||||||
|
} catch (e) {
|
||||||
|
// Fallback 1: Try scrollTo without behavior
|
||||||
|
try {
|
||||||
|
window.scrollTo(0, offsetPosition);
|
||||||
|
} catch (e2) {
|
||||||
|
// Fallback 2: Use element.scrollIntoView
|
||||||
|
const yOffset = -headerOffset;
|
||||||
|
const y = element.getBoundingClientRect().top + window.pageYOffset + yOffset;
|
||||||
|
window.scrollTo({ top: y, behavior: 'smooth' });
|
||||||
|
}
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
const handleMenuClick = (itemId: string) => {
|
const handleMenuClick = (itemId: string) => {
|
||||||
if (itemId === 'recipes') {
|
if (itemId === 'recipes') {
|
||||||
const isOnRecipePage = location.pathname === '/recipe';
|
const isOnRecipePage = location.pathname === '/recipe';
|
||||||
@@ -119,10 +140,9 @@ function Header({ showMenu = false }: HeaderProps) {
|
|||||||
window.location.href = '/recipe';
|
window.location.href = '/recipe';
|
||||||
return;
|
return;
|
||||||
} else {
|
} else {
|
||||||
|
// Scroll to section (drawer will be closed inside scrollToSection)
|
||||||
scrollToSection(itemId);
|
scrollToSection(itemId);
|
||||||
}
|
}
|
||||||
|
|
||||||
setIsDrawerOpen(false);
|
|
||||||
};
|
};
|
||||||
|
|
||||||
const menuItems = [
|
const menuItems = [
|
||||||
|
|||||||
Reference in New Issue
Block a user