fix broken edit links

primer · Dec 1, 2023 · 4cb27ae · 4cb27ae
1 parent 0b93ec2
commit 4cb27ae
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/packages/site/next.config.js b/packages/site/next.config.js
@@ -6,7 +6,8 @@ module.exports = {
     basePath: process.env.GITHUB_ACTIONS === 'true' ? '/nextocat' : '',
     publicRuntimeConfig: {
       siteTitle: 'Primer Nextocat',
-      docsRepositoryBase: 'https://github.com/primer/nextocat',
+      repo: 'https://github.com/primer/nextocat',
+      repoSrcPath: 'packages/site', // folder path to your site root. Helpful for monorepos.
       sidebarLinks: [
         {
           title: 'GitHub',

diff --git a/packages/theme/components/layout/sidebar/Sidebar.tsx b/packages/theme/components/layout/sidebar/Sidebar.tsx
@@ -90,9 +90,9 @@ export function Sidebar({pageMap}: SidebarProps) {
         if (item.kind === 'Folder') {
           const indexPage = item.children.find(child => child.kind === 'MdxPage' && child.name === 'index') as MdxFile
           const subNavName = indexPage.frontMatter.title
-          const shouldHideTabbedPages = indexPage.frontMatter['show-tabs'] || false
+          const shouldShowTabs = indexPage.frontMatter['show-tabs'] || false
 
-          if (shouldHideTabbedPages) {
+          if (shouldShowTabs) {
             return (
               <NavList.Item key={indexPage.name} href={`${basePath}${indexPage.route}`}>
                 {(indexPage as MdxFile).frontMatter?.title || item.name}
@@ -120,7 +120,7 @@ export function Sidebar({pageMap}: SidebarProps) {
 
                     return (
                       <NavList.Item
-                        key={(landingPageItem as MdxFile).name}
+                        key={`${(landingPageItem as MdxFile).route}`}
                         href={`${basePath}${(landingPageItem as MdxFile).route}`}
                         sx={{textTransform: 'capitalize'}}
                       >

diff --git a/packages/theme/index.tsx b/packages/theme/index.tsx
@@ -175,7 +175,11 @@ export default function Layout({children, pageOpts}: NextraThemeLayoutProps) {
                               <Stack direction="horizontal" padding="none" alignItems="center" gap={8}>
                                 <PencilIcon size={16} fill="var(--brand-InlineLink-color-rest)" />
 
-                                <InlineLink href={`https://github.com/primer/nextocat/blob/main/${filePath}`}>
+                                <InlineLink
+                                  href={`${publicRuntimeConfig.repo}/blob/main/${
+                                    publicRuntimeConfig.repoSrcPath ? `${publicRuntimeConfig.repoSrcPath}/` : ''
+                                  }${filePath}`}
+                                >
                                   Edit this page
                                 </InlineLink>
                               </Stack>