From 43cc641d1d1a74330ec5da454c284e280e94cca7 Mon Sep 17 00:00:00 2001
From: deme <joaommdemenech@gmail.com>
Date: Tue, 18 Apr 2023 08:27:46 -0300
Subject: [PATCH] [#754,landing page][s]: fix build errors

---
 site/components/Features.tsx | 7 +++++--
 site/components/Layout.tsx   | 4 ++--
 site/components/NavItem.tsx  | 6 ++----
 site/pages/[...slug].tsx     | 5 +++--
 4 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/site/components/Features.tsx b/site/components/Features.tsx
index c0c58f73..d1679550 100644
--- a/site/components/Features.tsx
+++ b/site/components/Features.tsx
@@ -47,8 +47,11 @@ export default function Features() {
         CKAN.
       </p>
       <div className="not-prose my-12 grid grid-cols-1 gap-6 md:grid-cols-2 lg:grid-cols-3">
-        {features.map((feature) => (
-          <div className="group relative rounded-xl border border-slate-200 dark:border-slate-800">
+        {features.map((feature, i) => (
+          <div
+            key={`feature-${i}`}
+            className="group relative rounded-xl border border-slate-200 dark:border-slate-800"
+          >
             <div className="absolute -inset-px rounded-xl border-2 border-transparent opacity-0 [background:linear-gradient(var(--quick-links-hover-bg,theme(colors.sky.50)),var(--quick-links-hover-bg,theme(colors.sky.50)))_padding-box,linear-gradient(to_top,theme(colors.blue.300),theme(colors.blue.400),theme(colors.blue.500))_border-box] group-hover:opacity-100 dark:[--quick-links-hover-bg:theme(colors.slate.800)]" />
             <div className="relative overflow-hidden rounded-xl p-6">
               <img src={feature.icon} alt="" className="h-24 w-auto" />
diff --git a/site/components/Layout.tsx b/site/components/Layout.tsx
index 836737ab..9d1b9195 100644
--- a/site/components/Layout.tsx
+++ b/site/components/Layout.tsx
@@ -57,7 +57,7 @@ export default function Layout({
   title?: string;
   tableOfContents?;
 }) {
-  const { toc } = children.props;
+  // const { toc } = children.props;
 
   const currentSection = useTableOfContents(tableOfContents);
 
@@ -94,7 +94,7 @@ export default function Layout({
         </a>
       </footer>
       {/** TABLE OF CONTENTS */}
-      {tableOfContents.length > 0 && (toc ?? siteConfig.tableOfContents) && (
+      {tableOfContents.length > 0 && (siteConfig.tableOfContents) && (
         <div className="hidden xl:fixed xl:right-0 xl:top-[4.5rem] xl:block xl:w-1/5 xl:h-[calc(100vh-4.5rem)] xl:flex-none xl:overflow-y-auto xl:py-16 xl:pr-6 xl:mb-16">
           <nav aria-labelledby="on-this-page-title" className="w-56">
             <h2 className="font-display text-md font-medium text-slate-900 dark:text-white">
diff --git a/site/components/NavItem.tsx b/site/components/NavItem.tsx
index 6aafe687..88d4dd93 100644
--- a/site/components/NavItem.tsx
+++ b/site/components/NavItem.tsx
@@ -58,15 +58,13 @@ export default function NavItem({ item }) {
             onMouseLeave={closeDropdown}
           >
             {item.subItems.map((subItem) => (
-              //  TODO: check the onClick error below
-              //  onClick does not exist on Menu.Item
               <Menu.Item
                 key={subItem.name}
-                onClick={() => setshowDropdown(false)}
-              >
+                >
                 <BaseLink
                   href={subItem.href}
                   className="text-slate-500 inline-flex items-center mt-2 px-1 pt-1 text-sm font-medium hover:text-slate-600"
+                  onClick={() => setshowDropdown(false)}
                 >
                   {subItem.name}
                 </BaseLink>
diff --git a/site/pages/[...slug].tsx b/site/pages/[...slug].tsx
index b3d4dfc2..db5bd5e0 100644
--- a/site/pages/[...slug].tsx
+++ b/site/pages/[...slug].tsx
@@ -19,12 +19,13 @@ export default function DRDPage({ source, frontMatter }) {
   const [tableOfContents, setTableOfContents] = useState([]);
 
   useEffect(() => {
-    const headingNodes = document.querySelectorAll('h2,h3');
+    const headingNodes = document.querySelectorAll(
+      'h2,h3'
+    ) as NodeListOf<HTMLHeadingElement>;
     const toc = collectHeadings(headingNodes);
     setTableOfContents(toc ?? []);
   }, [router.asPath]); // update table of contents on route change with next/link
 
-
   return (
     <Layout tableOfContents={tableOfContents} title={frontMatter.title}>
       <MDXPage source={source} frontMatter={frontMatter} />