pn-new-crm/src/components/shared/page-header.tsx

import { type ReactNode } from 'react';
import { cn } from '@/lib/utils';

interface PageHeaderProps {
  title: string;
  description?: string;
  actions?: ReactNode;
  className?: string;
  /** Optional small uppercase label above the title. */
  eyebrow?: string;
  /** Optional one-line stats / KPI summary under the description. */
  kpiLine?: ReactNode;
  /** Render with the polished gradient-brand-soft background strip. */
  variant?: 'plain' | 'gradient';
}

/**
 * Consistent page-level header: title, optional description, KPI sub-line,
 * eyebrow, and an action slot. Use `variant="gradient"` for hero strips on
 * landing pages and detail headers; the plain variant remains the default so
 * existing call-sites stay unchanged.
 *
 * Mobile-aware: below sm (640px) the title/eyebrow/description/gradient
 * frame all collapse - the page title is already shown by the mobile topbar,
 * so duplicating it in the body wastes scroll real estate. What remains is a
 * flush right-aligned action row (or nothing if there are no actions). On sm+
 * the full strip with title+description renders as before.
 */
export function PageHeader({
  title,
  description,
  actions,
  className,
  eyebrow,
  kpiLine,
  variant = 'plain',
}: PageHeaderProps) {
  const isGradient = variant === 'gradient';
  return (
    <>
      {/* Mobile: actions only. Title/description are duplicated by the topbar. */}
      {actions ? (
        <div className={cn('sm:hidden flex flex-wrap items-center justify-end gap-2', className)}>
          {actions}
        </div>
      ) : null}

      {/* Tablet + desktop: full strip with title, eyebrow, description, kpi line, actions.
          Stacks vertically at sm (640px) up to lg (1024px) so the title
          doesn't get truncated next to a wide actions row on tablet — the
          previous `sm:flex-row sm:flex-nowrap` forced four-button toolbars
          (e.g. the dashboard's DateRange + ExportPdf + Rearrange + Customize)
          onto one row at 768px, crushing the title. At lg+ the row layout
          returns. */}
      <div
        className={cn(
          // Removed `sm:mb-6` - the parent shell already provides
          // appropriate gap-y between header and the next section, and the
          // double-spacing produced an oversized top margin on dashboards.
          'hidden sm:flex flex-col gap-3 lg:flex-row lg:items-start lg:justify-between lg:gap-4',
          isGradient &&
            'rounded-xl border border-slate-200 bg-gradient-brand-soft px-5 py-4 shadow-xs',
          className,
        )}
        // Render the title element as <h1> for accessibility / SEO so screen
        // readers still get a heading on mobile from the topbar's <h1>.
        aria-hidden={undefined}
      >
        <div className="min-w-0 flex-1">
          {eyebrow ? (
            <div className="mb-1 text-xs font-semibold uppercase tracking-wide text-brand">
              {eyebrow}
            </div>
          ) : null}
          <h1 className="truncate text-xl font-bold tracking-tight text-foreground sm:text-2xl">
            {title}
          </h1>
          {description ? (
            <p className={cn('mt-1 text-sm text-muted-foreground')}>{description}</p>
          ) : null}
          {kpiLine ? (
            <div className="mt-2 flex flex-wrap items-center gap-x-4 gap-y-1 text-sm text-muted-foreground">
              {kpiLine}
            </div>
          ) : null}
        </div>
        {actions ? (
          <div className="flex shrink-0 flex-wrap items-center gap-2 lg:flex-nowrap">{actions}</div>
        ) : null}
      </div>
    </>
  );
}