Add hydration safety to AccountMenu
- Add mounted state to prevent hydration mismatches - Only render component after hydration completes - Close logout dialog when clicking outside - Prevent logout dialog from showing unexpectedly Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Haiku 4.5
parent
0cafa62713
commit
f5fbf1f338
@@ -9,13 +9,21 @@ type AccountMenuProps = {
|
|||||||
};
|
};
|
||||||
|
|
||||||
export default function AccountMenu({ customer }: AccountMenuProps) {
|
export default function AccountMenu({ customer }: AccountMenuProps) {
|
||||||
|
const [mounted, setMounted] = useState(false);
|
||||||
const [open, setOpen] = useState(false);
|
const [open, setOpen] = useState(false);
|
||||||
const [showLogoutConfirm, setShowLogoutConfirm] = useState(false);
|
const [showLogoutConfirm, setShowLogoutConfirm] = useState(false);
|
||||||
const ref = useRef<HTMLDivElement>(null);
|
const ref = useRef<HTMLDivElement>(null);
|
||||||
|
|
||||||
|
useEffect(() => {
|
||||||
|
setMounted(true);
|
||||||
|
}, []);
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
const onClickOutside = (e: MouseEvent) => {
|
const onClickOutside = (e: MouseEvent) => {
|
||||||
if (ref.current && !ref.current.contains(e.target as Node)) setOpen(false);
|
if (ref.current && !ref.current.contains(e.target as Node)) {
|
||||||
|
setOpen(false);
|
||||||
|
setShowLogoutConfirm(false);
|
||||||
|
}
|
||||||
};
|
};
|
||||||
document.addEventListener('mousedown', onClickOutside);
|
document.addEventListener('mousedown', onClickOutside);
|
||||||
return () => document.removeEventListener('mousedown', onClickOutside);
|
return () => document.removeEventListener('mousedown', onClickOutside);
|
||||||
@@ -29,6 +37,8 @@ export default function AccountMenu({ customer }: AccountMenuProps) {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (!mounted) return null;
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div ref={ref} className="relative">
|
<div ref={ref} className="relative">
|
||||||
<button
|
<button
|
||||||
@@ -54,7 +64,7 @@ export default function AccountMenu({ customer }: AccountMenuProps) {
|
|||||||
</div>
|
</div>
|
||||||
)}
|
)}
|
||||||
|
|
||||||
{showLogoutConfirm && (
|
{mounted && showLogoutConfirm && (
|
||||||
<div className="fixed inset-0 z-50 flex items-center justify-center bg-black/30">
|
<div className="fixed inset-0 z-50 flex items-center justify-center bg-black/30">
|
||||||
<div className="w-96 border border-line bg-paper p-6 shadow-lg">
|
<div className="w-96 border border-line bg-paper p-6 shadow-lg">
|
||||||
<h2 className="font-display text-lg">Log out?</h2>
|
<h2 className="font-display text-lg">Log out?</h2>
|
||||||
|
|||||||
Reference in New Issue
Block a user