Slight refactor

remix-run · Jul 1, 2024 · 9918a02 · 9918a02
1 parent 117b1fa
commit 9918a02
Showing 1 changed file with 19 additions and 17 deletions.
diff --git a/packages/remix-react/components.tsx b/packages/remix-react/components.tsx
@@ -200,12 +200,14 @@ function usePrefetchBehavior<T extends HTMLAnchorElement>(
 
 const ABSOLUTE_URL_REGEX = /^(?:[a-z][a-z0-9+.-]*:|\/\/)/i;
 
-function enableFogOfWarDiscovery(
+function getDiscoverAttr(
   discover: DiscoverBehavior,
   isAbsolute: boolean,
   reloadDocument: boolean | undefined
 ) {
-  return discover === "render" && !isAbsolute && !reloadDocument;
+  return discover === "render" && !isAbsolute && !reloadDocument
+    ? "true"
+    : undefined;
 }
 
 /**
@@ -230,11 +232,11 @@ let NavLink = React.forwardRef<HTMLAnchorElement, RemixNavLinkProps>(
           {...prefetchHandlers}
           ref={mergeRefs(forwardedRef, ref)}
           to={to}
-          data-discover={
-            enableFogOfWarDiscovery(discover, isAbsolute, props.reloadDocument)
-              ? "true"
-              : undefined
-          }
+          data-discover={getDiscoverAttr(
+            discover,
+            isAbsolute,
+            props.reloadDocument
+          )}
         />
         {shouldPrefetch && !isAbsolute ? (
           <PrefetchPageLinks page={href} />
@@ -269,11 +271,11 @@ let Link = React.forwardRef<HTMLAnchorElement, RemixLinkProps>(
           {...prefetchHandlers}
           ref={mergeRefs(forwardedRef, ref)}
           to={to}
-          data-discover={
-            enableFogOfWarDiscovery(discover, isAbsolute, props.reloadDocument)
-              ? "true"
-              : undefined
-          }
+          data-discover={getDiscoverAttr(
+            discover,
+            isAbsolute,
+            props.reloadDocument
+          )}
         />
         {shouldPrefetch && !isAbsolute ? (
           <PrefetchPageLinks page={href} />
@@ -302,11 +304,11 @@ let Form = React.forwardRef<HTMLFormElement, RemixFormProps>(
     return (
       <RouterForm
         {...props}
-        data-discover={
-          enableFogOfWarDiscovery(discover, isAbsolute, props.reloadDocument)
-            ? "true"
-            : undefined
-        }
+        data-discover={getDiscoverAttr(
+          discover,
+          isAbsolute,
+          props.reloadDocument
+        )}
       />
     );
   }