Skip to content

fix(web): keep settled thread notice compact #8980

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
t3-code wants to merge 2 commits into from
Closed

fix(web): keep settled thread notice compact #8980

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2f73930
fix(web): keep settled thread notice compact
t3-code[bot] Sep 1, 2026
8c54346
fix(web): separate compact notice copy for assistive tech
t3-code[bot] Sep 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 9 additions & 4 deletions apps/web/src/components/ChatView.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4962,14 +4962,19 @@ function ChatViewContent(props: ChatViewProps) {
return null;
}
const isSnoozed = activeThreadSnoozed;
const parkedThreadDescription = isSnoozed
? "Sending a message wakes it and moves it back to Active in the sidebar."
: "Send a message to unsettle";

@macroscopeapp macroscopeapp Bot Sep 1, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor consistency: the settled branch is now a terse fragment while the snoozed branch above stays a full sentence in the same slot, and it drops the "back to Active in the sidebar" cue that the sidebar state change relies on. It also says "unsettle" while the action button in the same row is labeled "Un-settle". Consider keeping both branches in one voice and terminology.

Suggested change
: "Send a message to unsettle";
: "Sending a message un-settles it and moves it back to Active in the sidebar.";

Posted via Macroscope — UI Consistency

return {
id: `thread-${isSnoozed ? "snoozed" : "settled"}:${activeThread?.id ?? "unknown"}`,
variant: "info",
icon: isSnoozed ? <AlarmClockIcon /> : <CheckCircle2Icon />,
title: `This thread is ${isSnoozed ? "snoozed" : "settled"}`,
description: isSnoozed
? "Sending a message wakes it and moves it back to Active in the sidebar."
: "Sending a message moves it back to Active in the sidebar.",
title: (
Comment thread
macroscopeapp[bot] marked this conversation as resolved.
<>
<span>This thread is {isSnoozed ? "snoozed" : "settled"}</span>{" "}
<span className="font-normal text-muted-foreground">{parkedThreadDescription}</span>
</>
),
Comment on lines +4972 to +4977

@macroscopeapp macroscopeapp Bot Sep 1, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inlining the detail as a bare fragment drops the one-line guards the other inline-detail banner titles use. ComposerBanner.Content is flex items-center gap-1 with min-w-0 on itself only, so these two spans keep their min-width: auto floor and the long snoozed sentence wraps inside the header row next to the Wake now button instead of staying a single-line title (the description slot it replaced rendered in its own ComposerBanner.Children row aligned under the icon column). The branch-mismatch item in this same file wraps its inline detail in flex min-w-0 items-baseline gap-1.5 with shrink-0 on the label and min-w-0 truncate on the trailing text; matching that keeps the row height and action alignment stable.

Suggested change
title: (
<>
<span>This thread is {isSnoozed ? "snoozed" : "settled"}</span>{" "}
<span className="font-normal text-muted-foreground">{parkedThreadDescription}</span>
</>
),
title: (
<span className="flex min-w-0 items-baseline gap-1.5">
<span className="shrink-0">This thread is {isSnoozed ? "snoozed" : "settled"}</span>
<span className="min-w-0 truncate font-normal text-muted-foreground">
{parkedThreadDescription}
</span>
</span>
),

Posted via Macroscope — UI Consistency

actions: (
<Button
size="xs"
Expand Down
Loading