[docs] Fix failing client-side navigation for /api routes (mui#28356)

gpvidal · Sep 15, 2021 · 5a0e7d9 · 5a0e7d9
1 parent 2503a09
commit 5a0e7d9
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/docs/next.config.js b/docs/next.config.js
@@ -165,7 +165,6 @@ module.exports = {
       },
     };
   },
-  trailingSlash: true,
   env: {
     COMMIT_REF: process.env.COMMIT_REF,
     ENABLE_AD: process.env.ENABLE_AD,
@@ -223,8 +222,13 @@ module.exports = {
     return map;
   },
   reactStrictMode,
+  trailingSlash: true,
   async rewrites() {
-    return [{ source: `/:lang(${LANGUAGES.join('|')})?/:rest*`, destination: '/:rest*' }];
+    return [
+      { source: `/:lang(${LANGUAGES.join('|')})?/:rest*`, destination: '/:rest*' },
+      // Make sure to include the trailing slash if `trailingSlash` option is set
+      { source: '/api/:rest*/', destination: '/api-docs/:rest*/' },
+    ];
   },
   // Can be turned on when https://github.com/vercel/next.js/issues/24640 is fixed
   optimizeFonts: false,

diff --git a/docs/src/modules/components/MarkdownLinks.js b/docs/src/modules/components/MarkdownLinks.js
@@ -51,15 +51,12 @@ function handleClick(event) {
   handleEvent(event, activeElement.getAttribute('href'));
 }
 
-let bound = false;
-
 export default function MarkdownLinks() {
   React.useEffect(() => {
-    if (bound) {
-      return;
-    }
-    bound = true;
     document.addEventListener('click', handleClick);
+    return () => {
+      document.addEventListener('click', handleClick);
+    };
   }, []);
 
   return null;