feat(channel): quill ThreadItem feed — day separators, smoother scroll#3265
Merged
Conversation
Rework the task card status row in the channel feed:
- Once a PR exists, its state (Merged / PR ready / Draft / Closed) is the
sole top-line status, replacing the run badge — so a shipped task never
reads "Ready + Merged" or a stale "In progress + PR ready".
- Merged PRs show a purple badge and lift the card to a purple border +
tint, matching the sidebar's merge accent.
- Fall back to the neutral "PR ready" when a PR URL exists but its GitHub
state hasn't resolved yet, so the badge and the "PR" link never disagree.
- "Ready" instead of "Completed" for finished runs — the work is ready to
look at, not necessarily shipped.
- Drop the redundant "Local" pill; environment shows in the meta row
("· Local" / "· Cloud").
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…roll
- Rebuild feed rows on quill's ThreadItem primitives (gutter avatar, header,
body, hover action bar) instead of hand-rolled ChatMessage markup.
- Group rows by day with a ChatMarker separator: "Today" / "Yesterday", then a
weekday + ordinal ("Monday 5th"), adding month/year further back.
- Cut scroll jank: memoize rows so task-list polls don't re-render the whole
feed, gate each row's 15s reply-teaser poll behind an in-view observer, and
tune contain-intrinsic-size to the real ~13rem row height.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
😎 Merged manually by @adamleithp - details. |
|
React Doctor found no issues in the changed files. 🎉 Reviewed by React Doctor for commit |
adamleithp
added
Stamphog
![stamphog[bot]](https://avatars.githubusercontent.com/in/3000332?s=60&v=4){kind=small}
stamphog
Bot
removed
the
Stamphog
Contributor
|
Reviews (1): Last reviewed commit: "feat(channel): quill ThreadItem feed wit..." | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
k11kirky
approved these changes
Jul 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rebuilds the channel feed on quill's ThreadItem primitives, adds day grouping, and cuts scroll jank.
Changes
ThreadItem(gutter avatar, header, body, hover action bar) instead of hand-rolledChatMessagemarkup. Needs the@posthog/quill0.3.0-beta.22bump (included).ChatMarkerbetween days: "Today" / "Yesterday", then weekday + ordinal ("Monday 5th"), adding month/year further back.· Local/· Cloud), dropping the redundant "Local" pill.contain-intrinsic-sizetuned to the real ~13rem row height.--muted-foreground).Supersedes #3254 (same work; that branch was renamed).