Tighten message density#1426
Conversation
|
🤖 ## Snapshot Compact message density with reactions and the thread-summary avatar stack.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.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: ab9fdfc0aa
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ceef5f95b6
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a91cda6e7a
ℹ️ 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".
…nsity-polish # Conflicts: # desktop/src/features/messages/ui/MessageThreadSummaryRow.tsx # desktop/src/features/messages/ui/TimelineMessageList.tsx # desktop/tests/e2e/messaging.spec.ts
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9547290be9
ℹ️ 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".
2 participants
Summary
Stack
Base:
kennylopez-thread-summary-alignmentHead:
kennylopez-message-density-polishSnapshots
Compact message density with reactions and the thread-summary avatar stack.
Validation
cd desktop && pnpm exec biome check src/features/messages/ui/MessageReactions.tsx src/features/messages/ui/MessageRow.tsx src/features/messages/ui/MessageThreadSummaryRow.tsx src/features/messages/ui/TimelineMessageList.tsxcd desktop && pnpm exec biome check tests/e2e/custom-emoji.spec.ts tests/e2e/messaging.spec.tscd desktop && pnpm check:px-textcd desktop && pnpm exec playwright test tests/e2e/custom-emoji.spec.ts tests/e2e/messaging.spec.ts --project=smokejust desktop-screenshot --name message-density-thread-reactions --active-channel general --messages /private/tmp/sprout-density-messages.json --clip 220,170,900,430 --viewport 1280x720 --outdir /private/tmp/sprout-density-shots --wait 2500just desktop-buildvia screenshot helperNote: local pre-push hooks were not rerun for the follow-up push because this machine is still nearly out of disk space; an earlier full pre-push attempt failed in Rust/Tauri test compilation with
No space left on devicewhile writing generated artifacts. The targeted desktop checks above passed.