BuzzbeeSCD/openchamber
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: temporarily disable hover timestamps in activity headers

Browse Source 
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
This commit is contained in:
Bohdan Triapitsyn
2026-03-04 03:03:27 +02:00
parent 5ddfc24221
commit 31c6d5e133
2 changed files with 12 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/ui/src/components/chat/message/parts/ReasoningPart.tsx
+6 -3
View File
@@ -83,6 +83,9 @@ const LiveDuration: React.FC<{ start: number; end?: number; active: boolean }> =
return <>{formatDuration(start, end, now)}</>; 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 = { type ReasoningTimelineBlockProps = {
text: string; text: string;
variant: ReasoningVariant; variant: ReasoningVariant;
@@ -165,7 +168,7 @@ export const ReasoningTimelineBlock: React.FC<ReasoningTimelineBlockProps> = ({
<span <span
className={cn( className={cn(
'text-muted-foreground/80 transition-opacity duration-150', 'text-muted-foreground/80 transition-opacity duration-150',
!isMobile && endedTimestampText && 'group-hover/tool:opacity-0' !isMobile && endedTimestampText && ENABLE_REASONING_HEADER_TIMESTAMPS && 'group-hover/tool:opacity-0'
)} )}
> >
<LiveDuration <LiveDuration
@@ -174,7 +177,7 @@ export const ReasoningTimelineBlock: React.FC<ReasoningTimelineBlockProps> = ({
active={typeof timeEnd !== 'number'} active={typeof timeEnd !== 'number'}
/> />
</span> </span>
{!isMobile && endedTimestampText ? ( {!isMobile && endedTimestampText && ENABLE_REASONING_HEADER_TIMESTAMPS ? (
<span <span
className={cn( className={cn(
'pointer-events-none absolute right-0 top-0 whitespace-nowrap text-muted-foreground/70 transition-opacity duration-150', 'pointer-events-none absolute right-0 top-0 whitespace-nowrap text-muted-foreground/70 transition-opacity duration-150',
@@ -186,7 +189,7 @@ export const ReasoningTimelineBlock: React.FC<ReasoningTimelineBlockProps> = ({
) : null} ) : null}
</span> </span>
) : null} ) : null}
{typeof timeStart !== 'number' && !isMobile && endedTimestampText ? ( {typeof timeStart !== 'number' && !isMobile && endedTimestampText && ENABLE_REASONING_HEADER_TIMESTAMPS ? (
<span className="text-muted-foreground/70 flex-shrink-0 tabular-nums"> <span className="text-muted-foreground/70 flex-shrink-0 tabular-nums">
{endedTimestampText} {endedTimestampText}
</span> </span>
packages/ui/src/components/chat/message/parts/ToolPart.tsx
+6 -3
View File
@@ -141,6 +141,9 @@ const LiveDuration: React.FC<{ start: number; end?: number; active: boolean }> =
return <>{formatDuration(start, end, now)}</>; 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<string, unknown>): { added: number; removed: number } | null => { const parseDiffStats = (metadata?: Record<string, unknown>): { added: number; removed: number } | null => {
if (!metadata?.diff || typeof metadata.diff !== 'string') return null; if (!metadata?.diff || typeof metadata.diff !== 'string') return null;
@@ -1823,7 +1826,7 @@ const ToolPart: React.FC<ToolPartProps> = ({
<span <span
className={cn( className={cn(
'text-muted-foreground/80 transition-opacity duration-150', 'text-muted-foreground/80 transition-opacity duration-150',
!isMobile && endedTimestampText && 'group-hover/tool:opacity-0' !isMobile && endedTimestampText && ENABLE_TOOL_HEADER_TIMESTAMPS && 'group-hover/tool:opacity-0'
)} )}
> >
<LiveDuration <LiveDuration
@@ -1832,7 +1835,7 @@ const ToolPart: React.FC<ToolPartProps> = ({
active={Boolean(isActive && typeof effectiveTimeEnd !== 'number')} active={Boolean(isActive && typeof effectiveTimeEnd !== 'number')}
/> />
</span> </span>
{!isMobile && endedTimestampText ? ( {!isMobile && endedTimestampText && ENABLE_TOOL_HEADER_TIMESTAMPS ? (
<span <span
className={cn( className={cn(
'pointer-events-none absolute right-0 top-0 whitespace-nowrap text-muted-foreground/70 transition-opacity duration-150', 'pointer-events-none absolute right-0 top-0 whitespace-nowrap text-muted-foreground/70 transition-opacity duration-150',
@@ -1844,7 +1847,7 @@ const ToolPart: React.FC<ToolPartProps> = ({
) : null} ) : null}
</span> </span>
) : null} ) : null}
{typeof effectiveTimeStart !== 'number' && !isMobile && endedTimestampText ? ( {typeof effectiveTimeStart !== 'number' && !isMobile && endedTimestampText && ENABLE_TOOL_HEADER_TIMESTAMPS ? (
<span className="ml-auto text-muted-foreground/70 flex-shrink-0 tabular-nums"> <span className="ml-auto text-muted-foreground/70 flex-shrink-0 tabular-nums">
{endedTimestampText} {endedTimestampText}
</span> </span>