clean up some commented code

Author: kmcfaul

src/components/NavSection.tsx

@@ -27,12 +27,7 @@ export const NavSection = ({
     navItems = [
       ...sortedNavEntries
         .reduce((map, entry) => {
-          if (
-            !map.has(entry.data.id) ||
-            (entry.data?.tab === 'react' &&
-              map.has(entry.data.id) &&
-              map.get(entry.data.id).data.tab === 'html')
-          ) {
+          if (!map.has(entry.data.id)) {
             map.set(entry.data.id, entry)
           }
           return map

src/content.config.ts

@@ -14,9 +14,10 @@ function defineContent(contentObj: CollectionDefinition) {
     return
   }
 
+  // Todo: expand to cover deprecated & demos tabs (look in filepath, or enforce tab in frontmatter for nonstandard tabs)
   const tabMap: any = {
-    'react-component-docs': 'react', // look in filepath for demos, look for deprecated flag or source flag for dep
-    'core-component-docs': 'html' // or force tab in frontmatter for non-base tabs
+    'react-component-docs': 'react',
+    'core-component-docs': 'html' 
   };
 
   return defineCollection({

src/content.ts

@@ -4,16 +4,10 @@ export const content = [
     "packageName":"@patternfly/react-core",
     "pattern":"**/components/**/*.md",
     "name":"react-component-docs"
-    // "build": {
-    //   "format": "directory"
-    // }
   }, 
   {
     "packageName":"@patternfly/patternfly",
     "pattern":"**/components/**/*.md",
     "name":"core-component-docs"
-    // "build": {
-    //   "format": "directory"
-    // }
   }
 ]

src/pages/[section]/[page]/[...tab].astro

@@ -10,6 +10,7 @@ export async function getStaticPaths() {
   const collections = await Promise.all(content.map(async (entry) => await getCollection(entry.name as 'textContent')))
 
   const flatCol =  collections.flat().map((entry) => {
+    // Build tabs dictionary
     // if no dictionary entry exists, and tab data exists
     if(componentTabs[entry.data.id] === undefined && entry.data.tab) { 
       componentTabs[entry.data.id] = [entry.data.tab];
@@ -48,6 +49,8 @@ export async function getStaticPaths() {
     return s1Index > s2Index ? 1 : -1;
   }
 
+  // Sort tabs entries based on above sort order
+  // Ensures all tabs are displayed in a consistent order & which tab gets displayed for a component route without a tab 
   Object.values(componentTabs).map((tabs: any) => {
     tabs.sort(sortSources)
   })