Skip to content

Conversation

@hristoterezov
Copy link
Member

@hristoterezov hristoterezov commented Oct 28, 2025

Summary

Adds dedicated toolbar buttons for polls and file sharing, plus a comprehensive disableChat configuration option.

Changes

New Toolbar Buttons

  • Added PollsButton and FileSharingButton in overflow menu
  • Buttons open chat panel with respective tab selected (like CC button)
  • Auto-hidden when features are disabled/unsupported

disableChat Configuration

  • New disableChat config option (default: false, whitelisted for overrides)
  • Disables: chat button, notifications, sounds, private messages, keyboard shortcuts
  • CHAT tab hidden when disabled, but polls/files/CC tabs remain accessible
  • Added getDefaultFocusedTab() to handle dynamic tab fallback

Test Plan

  • Polls button appears when supported
  • File sharing button appears when enabled
  • disableChat: true hides chat button and blocks access
  • Other tabs (polls/files/CC) remain accessible with chat disabled
  • Private message menus hidden when chat disabled
  • Keyboard shortcut 'c' blocked when chat disabled

@hristoterezov hristoterezov force-pushed the disable-chat branch 4 times, most recently from aea5d2d to f6c5289 Compare October 29, 2025 23:08
Adds dedicated buttons for polls and file sharing in the toolbar overflow menu, following the pattern of the CC button. Both buttons open the chat panel with their respective tab selected when clicked.
@hristoterezov hristoterezov force-pushed the disable-chat branch 2 times, most recently from fc6c379 to 412eb7e Compare October 30, 2025 18:12
Introduces a comprehensive disableChat config option that disables the entire chat feature including button visibility, notifications, sounds, private messages, and keyboard shortcuts. When disabled, the chat tab is hidden from the chat panel while allowing other tabs (polls, files, CC) to remain accessible.
@hristoterezov hristoterezov marked this pull request as ready for review October 30, 2025 21:20
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