fix: small minor errors (#1504)

* FIX: Map keys error * FIX: Duplicate key error * FIx: Animate presewnce where it is needed
code100x · Nov 12, 2024 · 88bff90 · 88bff90
1 parent 204c496
commit 88bff90
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 26 deletions.
diff --git a/src/components/BreadCrumbComponent.tsx b/src/components/BreadCrumbComponent.tsx
@@ -11,7 +11,7 @@ import {
 import { FullCourseContent } from '@/db/course';
 import { Home } from 'lucide-react';
 import Link from 'next/link';
-import { useMemo } from 'react';
+import { Fragment, useMemo } from 'react';
 
 export default function BreadCrumbComponent({
   rest,
@@ -93,7 +93,7 @@ export default function BreadCrumbComponent({
               finalRouteArray = [...rest];
             }
             return (
-              <>
+              <Fragment key={indexofBreadCrumbId}>
                 {index !== array.length - 1 ? (
                   <>
                     <BreadcrumbItem>
@@ -116,7 +116,7 @@ export default function BreadCrumbComponent({
                     </BreadcrumbPage>
                   </BreadcrumbItem>
                 )}
-              </>
+              </Fragment>
             );
           })}
         </BreadcrumbList>

diff --git a/src/components/Navbar.tsx b/src/components/Navbar.tsx
@@ -41,7 +41,7 @@ export const Navbar = () => {
   );
 
   return (
-    <AnimatePresence>
+    <>
       <motion.nav
         initial={{ opacity: 0, y: -20 }}
         animate={{ opacity: 1, y: 0 }}
@@ -143,35 +143,33 @@ export const Navbar = () => {
         </div>
 
         {/* Mobile menu */}
-        {!session?.user && (
-          <>
-            {isMenuOpen && (
+        <AnimatePresence>
+          {!session?.user && isMenuOpen && (
+            <motion.div
+              className="bg-background p-4 md:hidden"
+              initial={{ opacity: 0, y: -20 }}
+              animate={{ opacity: 1, y: 0 }}
+              exit={{ opacity: 0, y: -20 }}
+            >
               <motion.div
-                className="bg-background p-4 md:hidden"
+                className="flex gap-2"
                 initial={{ opacity: 0, y: -20 }}
                 animate={{ opacity: 1, y: 0 }}
                 exit={{ opacity: 0, y: -20 }}
               >
-                <motion.div
-                  className="flex gap-2"
-                  initial={{ opacity: 0, y: -20 }}
-                  animate={{ opacity: 1, y: 0 }}
-                  exit={{ opacity: 0, y: -20 }}
-                >
-                  <AppbarAuth />
+                <AppbarAuth />
                   <Button variant={'branding'} className="w-full">
-                    <Link
+                  <Link
                       href={'https://harkirat.classx.co.in/new-courses'}
-                      target="_blank"
-                    >
-                      Join now
-                    </Link>
-                  </Button>
-                </motion.div>
+                    target="_blank"
+                  >
+                    Join now
+                  </Link>
+                </Button>
               </motion.div>
-            )}
-          </>
-        )}
+            </motion.div>
+          )}
+        </AnimatePresence>
       </motion.nav>
 
       {/* Mobile search overlay */}
@@ -195,6 +193,6 @@ export const Navbar = () => {
           </motion.div>
         )}
       </AnimatePresence>
-    </AnimatePresence>
+    </>
   );
 };