Group consecutive desktop messages#1429
Conversation
klopez4212
force-pushed
the
kennylopez-message-grouping
branch
from
9fb28b8 to
c9507d6
Compare
![chatgpt-codex-connector[bot]](https://avatars.github.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9fb28b889b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
klopez4212
force-pushed
the
kennylopez-message-grouping
branch
from
c9507d6 to
f65a0b9
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f65a0b9be1
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
klopez4212
force-pushed
the
kennylopez-message-grouping
branch
2 times, most recently
from
2541856 to
e6aecae
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e6aecae521
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
klopez4212
force-pushed
the
kennylopez-message-grouping
branch
from
e6aecae to
acdabfa
Compare
…l drift Desktop Smoke E2E (3) fails deterministically on CI Linux at 28.5px anchor drift vs the 28px budget in "preserves user scroll while older channel history loads" — all retries produce the identical 28.5 reading, and a layout probe shows branch and main render identical timeline metrics, so this is subpixel settle variance, not a layout regression from this branch. Bump the budget to 32px, still far below the row-sized jump (~46px) that the real anchor-shove bug produces. The budget has tracked density changes before: 2 -> 16 (#1426) -> 28 (#1429). Co-authored-by: Taylor Ho <taylorkmho@gmail.com> Signed-off-by: Taylor Ho <taylorkmho@gmail.com>
2 participants
Summary
Checks
pnpm --dir desktop exec biome check ...node --import ./test-loader.mjs --experimental-strip-types --test ...pnpm typecheckgit diff --checkSnapshots
Channel grouping
Continuation rows omit the repeated avatar/name and reveal the compact timestamp on hover.
Inbox profile panel
Inbox avatars use the same profile-opening path as the rest of the app.