Merge branch 'master' into tyler-dashboard-newbie

Author: i30101

frontend/components/navbar/Navbar.tsx

@@ -46,15 +46,12 @@ export const Navbar = () => {
               </div>
             )}
 
-            {createNewConfig && (
-              <NavbarActions
-                createNewText={createNewConfig.text}
-                createNewHref={createNewConfig.href}
-                mobileShowHamburger={showListingsTabs}
-                isMobileMenuOpen={isMobileMenuOpen}
-                onToggleMobileMenu={toggleMobileMenu}
-              />
-            )}
+            <NavbarActions
+              createNewConfig={createNewConfig}
+              mobileShowHamburger={showListingsTabs}
+              isMobileMenuOpen={isMobileMenuOpen}
+              onToggleMobileMenu={toggleMobileMenu}
+            />
           </div>
         </div>
       </div>

frontend/components/navbar/NavbarActions.tsx

@@ -6,16 +6,14 @@ import { Button } from "@/components/ui/button";
 import { UserProfileDropdown } from "./UserProfileDropdown";
 
 interface Props {
-  createNewText: string;
-  createNewHref: string;
+  createNewConfig: { text: string; href: string } | undefined;
   mobileShowHamburger: boolean;
   isMobileMenuOpen: boolean;
   onToggleMobileMenu: () => void;
 }
 
 export const NavbarActions = ({
-  createNewText,
-  createNewHref,
+  createNewConfig,
   mobileShowHamburger,
   isMobileMenuOpen,
   onToggleMobileMenu,
@@ -61,33 +59,33 @@ export const NavbarActions = ({
 
   return (
     <div className="flex items-center gap-2 sm:gap-3">
-      {/* desktop only new listing button */}
-      {
-        <Button
-          variant="outline"
-          className="border-primary text-primary hover:bg-primary hover:text-primary-foreground hidden gap-2 md:flex"
-          asChild
-        >
-          <Link href={createNewHref} aria-label={createNewText}>
-            <Plus className="h-4 w-4" />
-            <span>{createNewText}</span>
-          </Link>
-        </Button>
-      }
+      {createNewConfig && (
+        <>
+          {/* desktop only new listing button */}
+          <Button
+            variant="outline"
+            className="border-primary text-primary hover:bg-primary hover:text-primary-foreground hidden gap-2 md:flex"
+            asChild
+          >
+            <Link href={createNewConfig.href} aria-label={createNewConfig.text}>
+              <Plus className="h-4 w-4" />
+              <span>{createNewConfig.text}</span>
+            </Link>
+          </Button>
 
-      {/* mobile only new listing button (icon only) */}
-      {
-        <Button
-          variant="outline"
-          size="icon"
-          className="border-primary text-primary hover:bg-primary hover:text-primary-foreground md:hidden"
-          asChild
-        >
-          <Link href={createNewHref} aria-label={createNewText}>
-            <Plus className="h-4 w-4" />
-          </Link>
-        </Button>
-      }
+          {/* mobile only new listing button (icon only) */}
+          <Button
+            variant="outline"
+            size="icon"
+            className="border-primary text-primary hover:bg-primary hover:text-primary-foreground md:hidden"
+            asChild
+          >
+            <Link href={createNewConfig.href} aria-label={createNewConfig.text}>
+              <Plus className="h-4 w-4" />
+            </Link>
+          </Button>
+        </>
+      )}
 
       {/* notification bell */}
       <Button