Merge pull request #251 from storybookjs/fix-document-titles-2

Don't assign an `undefined` document title
storybookjs · Dec 5, 2024 · 291c201 · 291c201
2 parents 4252f21 + a7f91fe
commit 291c201
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 13 deletions.
diff --git a/apps/addon-catalog/app/(home)/tag/[...name]/page.tsx b/apps/addon-catalog/app/(home)/tag/[...name]/page.tsx
@@ -31,10 +31,15 @@ export const generateMetadata: GenerateMetaData = async ({ params }) => {
   const data = (await fetchTagDetailsData(tagName)) || {};
 
   if ('error' in data) return {};
-  const { displayName } = data;
+
+  const title = data.displayName ?? data.name;
 
   return {
-    title: displayName ? `${displayName} tag | Storybook integrations` : undefined,
+    ...(title
+      ? {
+          title: `${title} tag | Storybook integrations`,
+        }
+      : undefined),
   };
 };
 

diff --git a/apps/addon-catalog/app/[...addonName]/page.tsx b/apps/addon-catalog/app/[...addonName]/page.tsx
@@ -35,10 +35,14 @@ export const generateMetadata: GenerateMetaData = async ({ params }) => {
   const name = (await params).addonName;
   const addon = await getAddonFromName(name);
 
+  const title = addon?.displayName ?? addon?.name;
+
   return {
-    title: addon?.displayName
-      ? `${addon.displayName} | Storybook integrations`
-      : undefined,
+    ...(title
+      ? {
+          title: `${title} | Storybook integrations`,
+        }
+      : undefined),
   };
 };
 

diff --git a/apps/frontpage/app/docs/[...slug]/page.tsx b/apps/frontpage/app/docs/[...slug]/page.tsx
@@ -50,7 +50,7 @@ export const generateMetadata: GenerateMetaData = async ({ params }) => {
   );
 
   return {
-    title: page?.title ? `${page.title} | Storybook docs` : undefined,
+    title: page?.title ? `${page.title} | Storybook docs` : 'Storybook docs',
     alternates: {
       canonical: findPage?.canonical,
     },

diff --git a/apps/frontpage/app/docs/page.tsx b/apps/frontpage/app/docs/page.tsx
@@ -9,7 +9,7 @@ export async function generateMetadata(): Promise<Metadata> {
   const page = await getPageData([latestVersion.id], latestVersion);
 
   return {
-    title: page?.title ? `${page.title} | Storybook docs` : undefined,
+    title: page?.title ? `${page.title} | Storybook docs` : 'Storybook docs',
     alternates: {
       canonical: '/docs',
     },

diff --git a/apps/frontpage/app/recipes/[...name]/page.tsx b/apps/frontpage/app/recipes/[...name]/page.tsx
@@ -31,9 +31,7 @@ interface RecipeDetailsProps {
   params: Params;
 }
 
-async function getRecipeFromName(
-  addonName: string[],
-): Promise<
+async function getRecipeFromName(addonName: string[]): Promise<
   // TODO: More precise typing to avoid these omits
   | Omit<
       RecipeWithTagLinks,
@@ -55,10 +53,10 @@ export const generateMetadata: GenerateMetaData = async ({ params }) => {
   const name = (await params).name;
   const recipe = await getRecipeFromName(name);
 
+  const title = recipe?.displayName ?? recipe?.name;
+
   return {
-    title: recipe?.displayName
-      ? `${recipe.displayName} | Storybook recipes`
-      : undefined,
+    title: title ? `${title} | Storybook recipes` : 'Storybook recipes',
   };
 };