From 31c6d5e133242bb7bd6382c8490f9fc57991989a Mon Sep 17 00:00:00 2001 From: Bohdan Triapitsyn Date: Wed, 4 Mar 2026 03:03:27 +0200 Subject: [PATCH] chore: temporarily disable hover timestamps in activity headers Hides date/timestamp overlays on tool card headers Hides date/timestamp overlays on thinking and justification headers Keeps timing logic in place behind TODO-gated flags for future UX improvements --- .../src/components/chat/message/parts/ReasoningPart.tsx | 9 ++++++--- .../ui/src/components/chat/message/parts/ToolPart.tsx | 9 ++++++--- 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/packages/ui/src/components/chat/message/parts/ReasoningPart.tsx b/packages/ui/src/components/chat/message/parts/ReasoningPart.tsx index baa40aae..0e74274b 100644 --- a/packages/ui/src/components/chat/message/parts/ReasoningPart.tsx +++ b/packages/ui/src/components/chat/message/parts/ReasoningPart.tsx @@ -83,6 +83,9 @@ const LiveDuration: React.FC<{ start: number; end?: number; active: boolean }> = return <>{formatDuration(start, end, now)}; }; +// TODO: Re-enable reasoning/justification header timestamps after hover UX is redesigned. +const ENABLE_REASONING_HEADER_TIMESTAMPS = false; + type ReasoningTimelineBlockProps = { text: string; variant: ReasoningVariant; @@ -165,7 +168,7 @@ export const ReasoningTimelineBlock: React.FC = ({ = ({ active={typeof timeEnd !== 'number'} /> - {!isMobile && endedTimestampText ? ( + {!isMobile && endedTimestampText && ENABLE_REASONING_HEADER_TIMESTAMPS ? ( = ({ ) : null} ) : null} - {typeof timeStart !== 'number' && !isMobile && endedTimestampText ? ( + {typeof timeStart !== 'number' && !isMobile && endedTimestampText && ENABLE_REASONING_HEADER_TIMESTAMPS ? ( {endedTimestampText} diff --git a/packages/ui/src/components/chat/message/parts/ToolPart.tsx b/packages/ui/src/components/chat/message/parts/ToolPart.tsx index 5d9b8356..3ca5d287 100644 --- a/packages/ui/src/components/chat/message/parts/ToolPart.tsx +++ b/packages/ui/src/components/chat/message/parts/ToolPart.tsx @@ -141,6 +141,9 @@ const LiveDuration: React.FC<{ start: number; end?: number; active: boolean }> = return <>{formatDuration(start, end, now)}; }; +// TODO: Re-enable tool header timestamp display after hover UX is redesigned. +const ENABLE_TOOL_HEADER_TIMESTAMPS = false; + const parseDiffStats = (metadata?: Record): { added: number; removed: number } | null => { if (!metadata?.diff || typeof metadata.diff !== 'string') return null; @@ -1823,7 +1826,7 @@ const ToolPart: React.FC = ({ = ({ active={Boolean(isActive && typeof effectiveTimeEnd !== 'number')} /> - {!isMobile && endedTimestampText ? ( + {!isMobile && endedTimestampText && ENABLE_TOOL_HEADER_TIMESTAMPS ? ( = ({ ) : null} ) : null} - {typeof effectiveTimeStart !== 'number' && !isMobile && endedTimestampText ? ( + {typeof effectiveTimeStart !== 'number' && !isMobile && endedTimestampText && ENABLE_TOOL_HEADER_TIMESTAMPS ? ( {endedTimestampText}