From 91562e652369c4e0df96dce7b96311d71bd105d6 Mon Sep 17 00:00:00 2001
From: Sam Becker <sam@sambecker.com>
Date: Fri, 11 Jul 2025 08:53:36 -0500
Subject: [PATCH] Fix tooltip button/form interaction

---
 src/app/Nav.tsx                                | 2 +-
 src/components/image/ZoomControls.tsx          | 1 +
 src/components/more/MoreMenu.tsx               | 1 +
 src/components/primitives/TooltipPrimitive.tsx | 1 +
 src/photo/InfinitePhotoScroll.tsx              | 1 +
 src/recipe/PhotoRecipeOverlayButton.tsx        | 1 +
 6 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/app/Nav.tsx b/src/app/Nav.tsx
index 5672f64c..61baa030 100644
--- a/src/app/Nav.tsx
+++ b/src/app/Nav.tsx
@@ -54,7 +54,7 @@ export default function Nav({
   ) =>
     typeof linkOrAction === 'string'
       ? <Link href={linkOrAction}>{text}</Link>
-      : <button onClick={linkOrAction}>{text}</button>;
+      : <button onClick={linkOrAction} type="button">{text}</button>;
 
   const switcherSelectionForPath = (): SwitcherSelection | undefined => {
     if (pathname === PATH_ROOT) {
diff --git a/src/components/image/ZoomControls.tsx b/src/components/image/ZoomControls.tsx
index 3c13a71e..2a61daff 100644
--- a/src/components/image/ZoomControls.tsx
+++ b/src/components/image/ZoomControls.tsx
@@ -41,6 +41,7 @@ export default function ZoomControls({
 
   const button = 
     <button
+      type="button"
       className={clsx(
         'fixed top-[20px] right-[70px]',
         'size-10 items-center justify-center',
diff --git a/src/components/more/MoreMenu.tsx b/src/components/more/MoreMenu.tsx
index 9296e7f4..de182c82 100644
--- a/src/components/more/MoreMenu.tsx
+++ b/src/components/more/MoreMenu.tsx
@@ -57,6 +57,7 @@ export default function MoreMenu({
     >
       <DropdownMenu.Trigger asChild>
         <button
+          type="button"
           className={clsx(
             'px-1 py-[3px]',
             'min-h-0 border-none shadow-none',
diff --git a/src/components/primitives/TooltipPrimitive.tsx b/src/components/primitives/TooltipPrimitive.tsx
index 45ded739..d33afdee 100644
--- a/src/components/primitives/TooltipPrimitive.tsx
+++ b/src/components/primitives/TooltipPrimitive.tsx
@@ -94,6 +94,7 @@ export default function TooltipPrimitive({
           {includeButton
             ? <button
               ref={refTrigger}
+              type="button"
               onClick={() => {
                 setIsOpen(!isOpen);
                 blurActiveElement();
diff --git a/src/photo/InfinitePhotoScroll.tsx b/src/photo/InfinitePhotoScroll.tsx
index 40151307..f3365248 100644
--- a/src/photo/InfinitePhotoScroll.tsx
+++ b/src/photo/InfinitePhotoScroll.tsx
@@ -159,6 +159,7 @@ export default function InfinitePhotoScroll({
   const renderMoreButton = () =>
     <div ref={buttonContainerRef}>
       <button
+        type="button"
         onClick={() => error ? mutate() : advance()}
         disabled={isLoading || isValidating}
         className={clsx(
diff --git a/src/recipe/PhotoRecipeOverlayButton.tsx b/src/recipe/PhotoRecipeOverlayButton.tsx
index b4b00f75..72a2daab 100644
--- a/src/recipe/PhotoRecipeOverlayButton.tsx
+++ b/src/recipe/PhotoRecipeOverlayButton.tsx
@@ -23,6 +23,7 @@ export default function PhotoRecipeOverlayButton({
     <Tooltip content={appText.tooltip.recipeInfo}>
       <button
         ref={ref}
+        type="button"
         onClick={() => {
           toggleRecipeOverlay?.();
           // Avoid unexpected tooltip trigger