Skip to content

Make the sidebar thread count expand followed threads - #23

Merged
itz4blitz merged 1 commit into
itz4blitz:mainfrom
vbmithr:fix/sidebar-thread-expansion
Oct 17, 2026
Merged

itz4blitz merged 1 commit into
itz4blitz:mainfrom
vbmithr:fix/sidebar-thread-expansion

Conversation

@vbmithr

@vbmithr vbmithr commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

Clicking “1 more thread” under a Discord channel previously did nothing: the sidebar emitted an inert count row. Make it expand the channel's followed threads inline, including read threads and unread threads beyond the three-row limit.

Add “Show fewer threads” to collapse them again, preserving the currently open thread. Remember expansion per network/account/channel for the session. Revealed threads use the ordinary conversation rows and open normally. Update the sidebar design specification.

Validation:

  • All nine sidebar-tree tests pass, including new regressions for one read thread, unread overflow, collapse with an open overflow thread, parent action keys, and account/channel isolation.
  • Arch package revision 20 builds successfully; all 18 local patches apply cleanly and match the build sources.
  • The user confirmed the fix works.
  • git diff --check passes.

Turn the inert thread count into an expansion toggle. Include read and
overflow followed threads, offer a collapse action, and keep the open
thread visible when folded. Scope expansion to the account and channel
for the session. Add regression coverage and update the design docs.

Co-Authored-By: GPT-6 <noreply@openai.com>

@itz4blitz itz4blitz left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

The count row was inert because Reveal::Threads opened nothing. It now toggles that channel's followed threads for the session, including read ones and those past the three-row cap, and "Show fewer threads" collapses them while the open thread stays. The key includes account, so another account's channel does not expand. Slack is left on its own path. Linux and macOS are green; I'll merge when Windows passes on 3389985.

@itz4blitz
itz4blitz merged commit c243cc1 into itz4blitz:main Oct 17, 2026
3 checks passed
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.

2 participants