fix: move DynamicFavicon to Sidebar to ensure it is present on all dashboard routes and subroutes

This commit is contained in:
Erik Silva
2026-01-21 15:35:06 -03:00
parent 2381e6e70f
commit 0ea18c5eaf
2 changed files with 1 additions and 2 deletions

View File

@@ -14,7 +14,6 @@ import {
} from "lucide-react"; } from "lucide-react";
import { Button } from "@/components/ui/button"; import { Button } from "@/components/ui/button";
import { Sidebar } from "@/components/Sidebar"; import { Sidebar } from "@/components/Sidebar";
import { DynamicFavicon } from "@/components/DynamicFavicon";
import { DynamicTitle } from "@/components/DynamicTitle"; import { DynamicTitle } from "@/components/DynamicTitle";
type Organization = { type Organization = {
@@ -58,7 +57,6 @@ export default function DashboardClient({
return ( return (
<div className="min-h-screen bg-white flex flex-col lg:flex-row"> <div className="min-h-screen bg-white flex flex-col lg:flex-row">
<DynamicFavicon logoUrl={organization.logoUrl} orgName={organization.name} primaryColor={primaryColor} />
<DynamicTitle title="Dashboard" orgName={organization.name} /> <DynamicTitle title="Dashboard" orgName={organization.name} />
<Sidebar user={user} organization={organization} /> <Sidebar user={user} organization={organization} />

View File

@@ -169,6 +169,7 @@ export function Sidebar({
return ( return (
<> <>
<DynamicFavicon logoUrl={organization.logoUrl} orgName={organization.name} primaryColor={primaryColor} />
{/* Mobile Header */} {/* Mobile Header */}
<div className="lg:hidden fixed top-0 left-0 right-0 h-16 bg-white border-b border-slate-200 z-40 flex items-center justify-between px-4"> <div className="lg:hidden fixed top-0 left-0 right-0 h-16 bg-white border-b border-slate-200 z-40 flex items-center justify-between px-4">
<div className="flex items-center gap-3"> <div className="flex items-center gap-3">