@handled-ai/design-system 0.20.13 → 0.20.14

package/src/components/comment-composer.tsx

@@ -55,29 +55,34 @@ function CommentComposer({
       data-slot="comment-composer"
       data-open={open ? "true" : undefined}
       className={cn(
-        "border-border bg-background flex items-start gap-2 rounded-lg border px-2 py-1.5 transition-colors",
-        open && "ring-ring/30 ring-2",
+        "flex items-start gap-4 rounded-xl transition-colors",
         className
       )}
     >
-      <Avatar size="sm" className="mt-px">
+      <Avatar size="sm" className="mt-1">
         {author?.avatarUrl ? <AvatarImage src={author.avatarUrl} alt={author.name ?? "You"} /> : null}
-        <AvatarFallback className="bg-muted text-muted-foreground text-[10px] font-medium uppercase">
+        <AvatarFallback className="bg-slate-700 text-[10px] font-semibold uppercase text-white dark:bg-slate-200 dark:text-slate-900">
           {getInitials({ name: author?.name, email: author?.email })}
         </AvatarFallback>
       </Avatar>
 
-      <div className="min-w-0 flex-1">
+      <div
+        data-slot="comment-composer-shell"
+        className={cn(
+          "min-w-0 flex-1 rounded-xl border border-border bg-background transition-[box-shadow,border-color]",
+          open ? "overflow-hidden shadow-sm" : "shadow-none"
+        )}
+      >
         <Textarea
           data-slot="comment-composer-input"
           value={text}
           onChange={(e) => setText(e.target.value)}
           onFocus={() => setFocused(true)}
           placeholder={placeholder}
-          rows={open ? 3 : 1}
+          rows={open ? 4 : 1}
           className={cn(
-            "resize-none border-0 bg-transparent px-1 py-0.5 text-sm leading-snug shadow-none focus-visible:ring-0",
-            !open && "min-h-0"
+            "resize-none rounded-none border-0 bg-transparent px-5 py-4 text-[15px] leading-6 shadow-none outline-none placeholder:text-muted-foreground/60 focus-visible:ring-0 focus-visible:ring-offset-0",
+            open ? "min-h-32" : "min-h-14"
           )}
           onKeyDown={(e) => {
             if ((e.metaKey || e.ctrlKey) && e.key === "Enter") {
@@ -88,15 +93,16 @@ function CommentComposer({
         />
 
         {open ? (
-          <div className="mt-0.5 flex items-center justify-between gap-2">
-            <span className="text-muted-foreground inline-flex items-center gap-1 text-[11px]">
-              <Lock size={12} /> {hint}
+          <div className="flex items-center justify-between gap-3 border-t border-border bg-muted/10 px-5 py-4">
+            <span className="inline-flex items-center gap-2 text-sm text-muted-foreground">
+              <Lock size={16} strokeWidth={1.75} /> {hint}
             </span>
-            <span className="flex items-center gap-2">
+            <span className="flex items-center gap-3">
               <Button
                 type="button"
                 variant="ghost"
                 size="sm"
+                className="px-2 text-sm font-medium text-muted-foreground hover:bg-transparent hover:text-foreground"
                 onClick={() => {
                   setText("")
                   setFocused(false)
@@ -104,9 +110,15 @@ function CommentComposer({
               >
                 Cancel
               </Button>
-              <Button type="button" size="sm" disabled={!canPost} onClick={post}>
+              <Button
+                type="button"
+                size="sm"
+                disabled={!canPost}
+                onClick={post}
+                className="rounded-lg bg-foreground px-4 text-sm font-semibold text-background shadow-none hover:bg-foreground/90"
+              >
                 Comment
-                <kbd className="bg-primary-foreground/15 ml-1 rounded px-1 text-[10px]">⌘↵</kbd>
+                <kbd className="ml-1 rounded px-1 text-[10px] text-background/70">⌘↵</kbd>
               </Button>
             </span>
           </div>

package/src/components/conversation-panel.tsx

@@ -1011,14 +1011,21 @@ function ConversationPanel({
   const draft = threads.filter((t) => effectiveStatus(t) === "draft").length
   const awaiting = threads.filter((t) => effectiveStatus(t) === "awaiting").length
   const anyPaused = threads.some((t) => t.paused)
-  const hubGmailThread = threads.find((t) => canOpenInGmail(t, onOpenInGmail))
+  const prioritizedThread =
+    threads.find((t) => t.status === "responded" && t.canReply !== false) ??
+    threads.find((t) => effectiveStatus(t) === "draft") ??
+    threads.find((t) => effectiveStatus(t) === "awaiting")
+  const hubGmailThread =
+    threads.find((t) => t.status === "responded" && t.canReply !== false && canOpenInGmail(t, onOpenInGmail)) ??
+    threads.find((t) => effectiveStatus(t) === "draft" && canOpenInGmail(t, onOpenInGmail)) ??
+    threads.find((t) => effectiveStatus(t) === "awaiting" && canOpenInGmail(t, onOpenInGmail)) ??
+    threads.find((t) => canOpenInGmail(t, onOpenInGmail))
   const firstAwaiting = threads.find((t) => effectiveStatus(t) === "awaiting")
 
   const [hubOpen, setHubOpen] = React.useState(true)
   const [openId, setOpenId] = React.useState<string | null>(() => {
     if (defaultOpenThreadId) return defaultOpenThreadId
-    const firstActionable = threads.find((t) => ["responded", "draft", "awaiting"].includes(t.status) && t.canReply !== false)
-    return firstActionable ? firstActionable.threadId : null
+    return prioritizedThread ? prioritizedThread.threadId : null
   })
 
   if (!threads.length) return null
@@ -1084,10 +1091,10 @@ function ConversationPanel({
             "inline-flex items-center gap-1.5 rounded-full px-2 py-0.5 text-[11px] font-semibold",
             responded > 0
               ? "bg-status-warning-bg text-status-warning-fg"
-              : awaiting > 0
-                ? "bg-status-info-bg text-status-info-fg"
-                : draft > 0
-                  ? "bg-status-pending-bg text-status-pending-fg"
+              : draft > 0
+                ? "bg-status-pending-bg text-status-pending-fg"
+                : awaiting > 0
+                  ? "bg-status-info-bg text-status-info-fg"
                   : "bg-muted text-muted-foreground"
           )}
         >

package/src/components/email-body.tsx

@@ -33,10 +33,55 @@ const PROSE = cn(
   "[&_sup]:align-super [&_sup]:text-[0.75em] [&_sub]:align-sub [&_sub]:text-[0.75em]",
 )
 
+const PLAIN_TEXT_LINK_RE = /https?:\/\/[^\s<>"']+|www\.[^\s<>"']+|[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,}/gi
+const TRAILING_LINK_PUNCTUATION_RE = /[),.;:!?]+$/
+
+function plainTextHref(value: string): string | null {
+  if (/^[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,}$/i.test(value)) {
+    return `mailto:${value}`
+  }
+  const href = value.toLowerCase().startsWith("www.") ? `https://${value}` : value
+  try {
+    const url = new URL(href)
+    return url.protocol === "http:" || url.protocol === "https:" ? href : null
+  } catch {
+    return null
+  }
+}
+
+function linkifyPlainText(text: string): React.ReactNode[] {
+  const nodes: React.ReactNode[] = []
+  let cursor = 0
+
+  for (const match of text.matchAll(PLAIN_TEXT_LINK_RE)) {
+    const value = match[0]
+    const index = match.index ?? 0
+    if (index > cursor) nodes.push(text.slice(cursor, index))
+
+    const trailing = value.match(TRAILING_LINK_PUNCTUATION_RE)?.[0] ?? ""
+    const linkText = trailing ? value.slice(0, -trailing.length) : value
+    const href = plainTextHref(linkText)
+    if (href) {
+      nodes.push(
+        <a key={`${index}-${linkText}`} href={href} target="_blank" rel="noreferrer noopener">
+          {linkText}
+        </a>,
+      )
+    } else {
+      nodes.push(linkText)
+    }
+    if (trailing) nodes.push(trailing)
+    cursor = index + value.length
+  }
+
+  if (cursor < text.length) nodes.push(text.slice(cursor))
+  return nodes
+}
+
 function PlainTextBlock({ text, className, slot }: { text: string; className?: string; slot: string }) {
   return (
     <div data-slot={slot} className={cn(PROSE, "whitespace-pre-line", className)}>
-      {decodeEmailDisplayText(text)}
+      {linkifyPlainText(decodeEmailDisplayText(text))}
     </div>
   )
 }

package/src/prototype/__tests__/detail-view-case-panel-v2.test.tsx

@@ -101,7 +101,7 @@ describe("DetailView case-panel-v2 section layout", () => {
       screen.getByText("Cash movement"),
       screen.getByText("Approve action"),
       screen.getByText("After-score marker"),
-      screen.getByText("Activity timeline"),
+      screen.getByText(/activity timeline/i),
       screen.getByText("Legacy detail extra marker"),
     )
   })
@@ -125,7 +125,7 @@ describe("DetailView case-panel-v2 section layout", () => {
       screen.getByText("Opportunity marker"),
       screen.getByText("Primary action marker"),
       screen.getByText("Comment area marker"),
-      screen.getByText("Activity timeline"),
+      screen.getByText(/activity timeline/i),
     )
   })
 
@@ -148,7 +148,7 @@ describe("DetailView case-panel-v2 section layout", () => {
 
     expectInDocumentOrder(
       screen.getByText("Comment composer marker"),
-      screen.getByText("Activity timeline"),
+      screen.getByText(/activity timeline/i),
     )
   })
 

package/src/prototype/__tests__/detail-view-timeline-system-events.test.tsx

@@ -209,20 +209,22 @@ describe("DetailView timeline system-events toggle", () => {
     const badge = container.querySelector('[data-testid="hidden-count-badge"]')
     expect(badge).not.toBeNull()
     expect(badge?.textContent).toBe("2")
-    expect(badge).toHaveClass("min-w-[18px]")
+    expect(badge).toHaveClass("min-w-[22px]")
   })
 
-  it("calls localStorage.setItem when toggle changes and shows a stronger pressed style", () => {
+  it("calls localStorage.setItem when toggle changes and shows the active pill style", () => {
     const { container } = render(<DetailView {...baseProps()} />)
     expandTimeline(container)
     const toggle = container.querySelector(
       '[data-testid="system-events-toggle"]',
     ) as HTMLElement
     expect(toggle).toHaveAttribute("aria-pressed", "false")
+    expect(toggle).toHaveAttribute("title", "Score changes are hidden.")
     fireEvent.click(toggle)
     expect(toggle).toHaveAttribute("aria-pressed", "true")
-    expect(toggle.className).toContain("border-primary/40")
-    expect(toggle.className).toContain("bg-primary/10")
+    expect(toggle).toHaveClass("border-foreground")
+    expect(toggle).toHaveClass("bg-foreground")
+    expect(toggle).toHaveAttribute("title", "Showing 2 score changes.")
     expect(localStorageMock.setItem).toHaveBeenCalledWith(
       "test-show-score-changes",
       "true",
@@ -312,18 +314,19 @@ describe("DetailView timeline system-events toggle", () => {
     expect(toggle).toBeNull()
   })
 
-  it("shows footer hint below the case-panel timeline when timeline is expanded and system events are hidden", () => {
+  it("does not render a footer hint and uses the hidden hint as toggle help", () => {
     const { container } = render(<DetailView {...baseProps()} />)
     expandTimeline(container)
     const timeline = container.querySelector('[data-variant="case-panel"]')
     const hint = container.querySelector('[data-testid="timeline-footer-hint"]')
+    const toggle = container.querySelector('[data-testid="system-events-toggle"]')
     expect(timeline).not.toBeNull()
-    expect(hint).not.toBeNull()
-    expect(hint?.textContent).toBe("Score changes are hidden.")
-    expect(timeline?.compareDocumentPosition(hint as Node)).toBe(Node.DOCUMENT_POSITION_FOLLOWING)
+    expect(hint).toBeNull()
+    expect(toggle).toHaveAttribute("title", "Score changes are hidden.")
+    expect(toggle).toHaveAttribute("aria-label", "Score changes are hidden.")
   })
 
-  it("shows visible footer hint with count when system events are shown", () => {
+  it("uses visible footer hint text as toggle help with count when system events are shown", () => {
     const { container } = render(<DetailView {...baseProps()} />)
     expandTimeline(container)
     // Toggle on
@@ -332,8 +335,8 @@ describe("DetailView timeline system-events toggle", () => {
     ) as HTMLElement
     fireEvent.click(toggle)
     const hint = container.querySelector('[data-testid="timeline-footer-hint"]')
-    expect(hint).not.toBeNull()
-    expect(hint?.textContent).toBe("Showing 2 score changes.")
+    expect(hint).toBeNull()
+    expect(toggle).toHaveAttribute("title", "Showing 2 score changes.")
   })
 
   // --- Toggle always renders when system-noise events exist (review fix #1) ---
@@ -414,10 +417,10 @@ describe("DetailView timeline system-events toggle", () => {
     const toggle = container.querySelector('[data-testid="system-events-toggle"]')
     expect(toggle).not.toBeNull()
     expect(toggle?.textContent).toContain("Legacy label")
-    // Footer hint should work too
+    // Deprecated hint props are accepted and exposed as toggle help.
     expandTimeline(container)
     const hint = container.querySelector('[data-testid="timeline-footer-hint"]')
-    expect(hint).not.toBeNull()
-    expect(hint?.textContent).toBe("Legacy hidden hint.")
+    expect(hint).toBeNull()
+    expect(toggle).toHaveAttribute("title", "Legacy hidden hint.")
   })
 })

package/src/prototype/prototype-inbox-view.tsx

@@ -222,6 +222,7 @@ function TimelineSection({
   attentionCount,
   sysEvtConfig,
   lastActivityTime,
+  isCasePanel = false,
 }: {
   timelineEvents: TimelineEvent[]
   showTimeline: boolean
@@ -231,6 +232,7 @@ function TimelineSection({
   attentionCount?: number
   sysEvtConfig?: TimelineSystemEventsConfig
   lastActivityTime?: string
+  isCasePanel?: boolean
 }) {
   // Single-pass partition: compute visibleEvents and hiddenCount together
   const visibleEvents: TimelineEvent[] = []
@@ -245,6 +247,9 @@ function TimelineSection({
   // config was provided — so consumers that emit `isSystemNoise: true` always
   // give users a way to reveal those events.
   const toggleLabel = sysEvtConfig?.toggleLabel ?? "System events"
+  const toggleHelp = showSystemEvents
+    ? sysEvtConfig?.visibleHint?.replace("{count}", String(hiddenCount)) ?? "Hide system events"
+    : sysEvtConfig?.hiddenHint ?? "Show system events"
 
   // Derive "Last activity" from the first *visible* event so the collapsed
   // header never points at a hidden score-update. The caller-supplied
@@ -262,84 +267,106 @@ function TimelineSection({
   const eventCountLabel = `${visibleCount} ${visibleCount === 1 ? "event" : "events"}`
 
   return (
-    <div className="mb-8">
+    <div
+      className={cn(
+        isCasePanel ? "mt-8 border-t border-border pt-8 pb-8" : "mb-8"
+      )}
+    >
       {/* Header — outer non-interactive container */}
       <div
-        className="group/timeline flex w-full items-center justify-between gap-2 py-2 rounded-md transition-colors hover:bg-muted/40 -mx-2 px-2"
+        className={cn(
+          "flex w-full items-center justify-between",
+          isCasePanel
+            ? "gap-4 border-b border-border pb-5"
+            : "group/timeline gap-2 rounded-md py-2 transition-colors hover:bg-muted/40 -mx-2 px-2"
+        )}
         data-testid="timeline-header"
       >
         {/* Left: collapse/expand button */}
         <button
           type="button"
           onClick={() => setShowTimeline((prev) => !prev)}
-          className="flex items-center gap-2 cursor-pointer bg-transparent border-0 p-0"
+          className="flex min-w-0 cursor-pointer items-center gap-2 border-0 bg-transparent p-0 text-left"
           data-testid="timeline-collapse-btn"
         >
-          <h3 className="text-xs font-bold text-muted-foreground uppercase tracking-wider group-hover/timeline:text-foreground transition-colors">Activity timeline</h3>
+          <h3 className="text-xs font-bold uppercase tracking-[0.16em] text-muted-foreground transition-colors group-hover/timeline:text-foreground">ACTIVITY TIMELINE</h3>
           {!showTimeline && attentionCount != null && attentionCount > 0 && (
             <span className="inline-flex items-center gap-1 rounded-full bg-destructive/10 px-1.5 py-0.5 text-[10px] font-semibold text-destructive border border-destructive/20">
               {attentionCount} new
             </span>
           )}
-          {!showTimeline && firstVisibleTime && (
+          {!isCasePanel && !showTimeline && firstVisibleTime && (
             <span className="text-[11px] text-muted-foreground/60" data-testid="last-activity-hint">
               &middot; Last activity {firstVisibleTime}
             </span>
           )}
-          <div className="flex items-center gap-1.5">
-            <span className="text-[11px] font-medium text-muted-foreground" data-testid="event-count">{eventCountLabel}</span>
-            <ChevronDown className={`h-3.5 w-3.5 text-muted-foreground transition-transform duration-200 ${showTimeline ? "rotate-180" : ""}`} />
-          </div>
         </button>
 
-        {/* Right: system-events toggle — always rendered when noise events exist */}
-        {hasSystemNoise && (
+        {/* Right: system-events toggle, event count, and collapse affordance */}
+        <div className="flex shrink-0 items-center gap-4">
+          {hasSystemNoise && (
+            <button
+              type="button"
+              onClick={() => setShowSystemEvents((prev) => !prev)}
+              className={cn(
+                "inline-flex shrink-0 cursor-pointer items-center gap-3 rounded-full border px-3.5 py-2 text-sm font-semibold transition-colors",
+                showSystemEvents
+                  ? "border-foreground bg-foreground text-background shadow-sm hover:bg-foreground/90"
+                  : "border-border bg-background text-muted-foreground shadow-sm hover:bg-muted/40 hover:text-foreground"
+              )}
+              aria-pressed={showSystemEvents}
+              aria-label={toggleHelp}
+              title={toggleHelp}
+              data-testid="system-events-toggle"
+            >
+              <span
+                className={cn(
+                  "relative inline-flex h-4 w-8 shrink-0 items-center rounded-full p-0.5 transition-colors",
+                  showSystemEvents ? "bg-teal-600" : "bg-muted-foreground/30"
+                )}
+                aria-hidden="true"
+                data-testid="system-events-indicator"
+              >
+                <span
+                  className={cn(
+                    "block h-3 w-3 rounded-full bg-white shadow-sm transition-transform",
+                    showSystemEvents ? "translate-x-4" : "translate-x-0"
+                  )}
+                />
+              </span>
+              <span>{toggleLabel}</span>
+              {!showSystemEvents ? (
+                <span
+                  className="inline-flex min-w-[22px] items-center justify-center rounded-full bg-muted px-1.5 text-xs font-bold tabular-nums text-muted-foreground"
+                  data-testid="hidden-count-badge"
+                >
+                  {hiddenCount}
+                </span>
+              ) : null}
+            </button>
+          )}
+
           <button
             type="button"
-            onClick={() => setShowSystemEvents((prev) => !prev)}
+            onClick={() => setShowTimeline((prev) => !prev)}
             className={cn(
-              "flex shrink-0 cursor-pointer items-center gap-1.5 rounded-full border px-2.5 py-1 text-[11px] font-medium transition-colors hover:text-foreground",
-              showSystemEvents
-                ? "border-primary/40 bg-primary/10 text-primary shadow-sm hover:bg-primary/15"
-                : "border-border bg-background text-muted-foreground hover:bg-muted/40"
+              "inline-flex shrink-0 cursor-pointer items-center border-0 bg-transparent p-0 text-muted-foreground transition-colors hover:text-foreground",
+              isCasePanel ? "gap-3 text-sm" : "gap-1.5 text-[11px]"
             )}
-            aria-pressed={showSystemEvents}
-            data-testid="system-events-toggle"
+            aria-label={showTimeline ? "Collapse activity timeline" : "Expand activity timeline"}
           >
-            {toggleLabel}
-            <span
-              className={cn(
-                "inline-flex min-w-[18px] items-center justify-center rounded-full px-1.5 text-[10px] font-semibold tabular-nums",
-                showSystemEvents
-                  ? "bg-primary/15 text-primary ring-1 ring-primary/30"
-                  : "bg-muted text-muted-foreground ring-1 ring-border/70"
-              )}
-              data-testid="hidden-count-badge"
-            >
-              {hiddenCount}
-            </span>
+            <span className="font-medium" data-testid="event-count">{eventCountLabel}</span>
+            <ChevronDown className={`h-3.5 w-3.5 transition-transform duration-200 ${showTimeline ? "rotate-180" : ""}`} />
           </button>
-        )}
+        </div>
       </div>
 
       {/* Timeline body */}
       {showTimeline && visibleEvents.length > 0 && (
-        <div className="mt-3">
+        <div className="mt-6">
           <TimelineActivity events={visibleEvents} variant="case-panel" />
         </div>
       )}
-
-      {/* Footer hint */}
-      {showTimeline && !showSystemEvents && sysEvtConfig?.hiddenHint && hasSystemNoise && (
-        <p className="mt-2 text-[11px] text-muted-foreground/60 border-t border-dashed border-border pt-2" data-testid="timeline-footer-hint">
-          {sysEvtConfig.hiddenHint}
-        </p>
-      )}
-      {showTimeline && showSystemEvents && sysEvtConfig?.visibleHint && hasSystemNoise && (
-        <p className="mt-2 text-[11px] text-muted-foreground/60 border-t border-dashed border-border pt-2" data-testid="timeline-footer-hint">
-          {sysEvtConfig.visibleHint.replace("{count}", String(hiddenCount))}
-        </p>
-      )}
     </div>
   )
 }
@@ -583,6 +610,7 @@ export function DetailView({
       attentionCount={attentionCount}
       sysEvtConfig={sysEvtConfig}
       lastActivityTime={lastActivityTime}
+      isCasePanel={isCasePanelV2}
     />
   ) : null