Skip to content

fix(desktop): clean up channel management sidebar layout#509

Merged
wesbillman merged 2 commits into
mainfrom
rick/channel-management-sidebar-cleanup
May 8, 2026
Merged

fix(desktop): clean up channel management sidebar layout#509
wesbillman merged 2 commits into
mainfrom
rick/channel-management-sidebar-cleanup

Conversation

@wesbillman
Copy link
Copy Markdown
Collaborator

Summary

  • Removed redundant section labels ("Details", "Topic and Purpose") and section descriptions throughout the manage channel sidebar
  • Fixed member count to use actual members query data instead of always showing 0
  • Moved canvas section to top of sidebar for quick access
  • Converted topic and purpose fields to single-line inputs
  • Consolidated bottom row layout: [Leave] [Archive] [spacer] [Delete] with delete right-aligned
  • Moved create workflow button from channel top bar into the sidebar
  • Removed the old Section component wrapper (no longer needed)

2 files changed (ChannelManagementSheet.tsx, ChannelMembersBar.tsx), net -73 lines.

Test plan

  • Open manage channel sidebar — verify no section labels like "Details" or "Topic and Purpose"
  • Verify member count shows actual count, not 0
  • Canvas section appears at the top of the sidebar content
  • Topic and purpose are single-line inputs (not textareas)
  • Bottom row shows [Leave] [Archive/Unarchive] on the left, [Delete] on the right
  • Create workflow button appears in sidebar, not in channel top bar
  • Archive/unarchive and delete still function correctly
  • Non-owners don't see the delete button
  • Non-members see "Join channel" instead of "Leave"

🤖 Generated with Claude Code

@wesbillman
Copy link
Copy Markdown
Collaborator Author

Basic fixes for now.

Screenshot 2026-05-08 at 9 23 39 AM Screenshot 2026-05-08 at 9 23 41 AM

@wesbillman wesbillman enabled auto-merge (squash) May 8, 2026 16:27
@wesbillman wesbillman force-pushed the rick/channel-management-sidebar-cleanup branch from f4152a8 to 3dc98b9 Compare May 8, 2026 16:53
wesbillman and others added 2 commits May 8, 2026 10:02
- Remove redundant section labels ("Details", "Topic and Purpose", etc.)
- Fix member count to use actual members query data
- Move canvas section to top of sidebar
- Convert topic/purpose to single-line inputs
- Move leave button to bottom row: [Leave] [Archive] [spacer] [Delete]
- Move create workflow button from top bar into sidebar
- Remove redundant section descriptions throughout
- Right-align delete button in danger zone row

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The manage channel sidebar no longer uses section headings, so the test
that verified canvas placement by checking section h2 order now fails.
Replace with a bounding-box assertion that canvas content renders above
the name input.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@wesbillman wesbillman force-pushed the rick/channel-management-sidebar-cleanup branch from 3dc98b9 to 8ed2474 Compare May 8, 2026 17:04
@wesbillman wesbillman merged commit a05577f into main May 8, 2026
15 checks passed
@wesbillman wesbillman deleted the rick/channel-management-sidebar-cleanup branch May 8, 2026 17:14
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