Skip to content

fix(desktop): show thread replies loader#1773

Merged
wesbillman merged 1 commit into
mainfrom
pinky/thread-initial-loader
Jul 12, 2026
Merged

fix(desktop): show thread replies loader#1773
wesbillman merged 1 commit into
mainfrom
pinky/thread-initial-loader

Conversation

@wesbillman

Copy link
Copy Markdown
Collaborator

Summary

  • keep the known thread head and composer visible while replies load
  • show existing skeleton rows in the reply region during the initial query
  • render the empty state only after a completed empty response

Validation

  • pnpm typecheck
  • pnpm exec biome check src/features/messages/ui/MessageThreadPanel.tsx src/features/channels/ui/ChannelPane.tsx src/features/channels/ui/ChannelPane.types.ts src/features/channels/ui/ChannelScreen.tsx
  • pnpm test (2,546 passed)
  • pnpm build
  • pnpm exec playwright test tests/e2e/messaging.spec.ts --grep "thread panel width uses session storage and reset handle" --project smoke --reporter=line

Keep the known thread head visible while the initial reply query loads,
and reserve the empty state for a completed empty response.

Co-authored-by: Pinky <44b8e82baa6e0e254e0208d68f335c283c94e7b78dd1fa10d5a49d3f13dd0435@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: Wes <wesbillman@users.noreply.github.com>
@wesbillman wesbillman requested a review from a team as a code owner July 12, 2026 16:13
@wesbillman

Copy link
Copy Markdown
Collaborator Author

@codex please review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Swish!

Reviewed commit: 665d1b60bc

ℹ️ 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".

@wesbillman wesbillman merged commit e07e02c into main Jul 12, 2026
25 checks passed
@wesbillman wesbillman deleted the pinky/thread-initial-loader branch July 12, 2026 16:27
wpfleger96 pushed a commit that referenced this pull request Jul 12, 2026
* origin/main:
  fix(desktop): correct provider and model handling in agent config dialogs (#1764)
  fix(desktop): cascade persona deletes and restart agents on global config save (#1766)
  fix(desktop): make doctor installs retryable with per-runtime progress and auth status (#1765)
  fix(codex): swap to @agentclientprotocol/codex-acp 1.x + detect outdated adapter (#1750)
  Fix oversized mobile Manage channel sheet (#1774)
  fix(desktop): show thread replies loader (#1773)
  fix(build): allow vendored opus to configure under CMake 4 (#1763)
  fix(desktop): restore multi-image mosaic galleries (#1769)
  Group channel membership events (#1713)
  Fix desktop launch motion and reaction spacing (#1717)
  fix(desktop): align sidebar search across themes (#1712)
  fix(desktop): Buzz theme flicker, white bar & accent-picker motion (#1681)
  fix(mobile): highlight full multi-word mentions (#1762)
  [BOT-1264] fix(mobile): #channel tags not tappable (#1695)
  feat(desktop): surface Team Instructions as distinct observer section (#1759)

# Conflicts:
#	desktop/playwright.config.ts
wpfleger96 pushed a commit that referenced this pull request Jul 12, 2026
* origin/main:
  fix(desktop): correct provider and model handling in agent config dialogs (#1764)
  fix(desktop): cascade persona deletes and restart agents on global config save (#1766)
  fix(desktop): make doctor installs retryable with per-runtime progress and auth status (#1765)
  fix(codex): swap to @agentclientprotocol/codex-acp 1.x + detect outdated adapter (#1750)
  Fix oversized mobile Manage channel sheet (#1774)
  fix(desktop): show thread replies loader (#1773)
  fix(build): allow vendored opus to configure under CMake 4 (#1763)
  fix(desktop): restore multi-image mosaic galleries (#1769)
  Group channel membership events (#1713)
  Fix desktop launch motion and reaction spacing (#1717)
  fix(desktop): align sidebar search across themes (#1712)
  fix(desktop): Buzz theme flicker, white bar & accent-picker motion (#1681)
  fix(mobile): highlight full multi-word mentions (#1762)
  [BOT-1264] fix(mobile): #channel tags not tappable (#1695)
  feat(desktop): surface Team Instructions as distinct observer section (#1759)

Co-authored-by: Will Pfleger <pfleger.will@gmail.com>
Signed-off-by: Will Pfleger <pfleger.will@gmail.com>

# Conflicts:
#	desktop/playwright.config.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant