Skip to content

Comments

feat: Toggle Chat Settings for Sidebar#2759

Merged
hayescode merged 1 commit intomainfrom
feat/settings_sidebar
Jan 18, 2026
Merged

feat: Toggle Chat Settings for Sidebar#2759
hayescode merged 1 commit intomainfrom
feat/settings_sidebar

Conversation

@hayescode
Copy link
Contributor

@hayescode hayescode commented Jan 17, 2026

Feature: Chat Settings Sidebar

This PR adds a configuration option to display chat settings in a dedicated sidebar instead of the default message composer modal.

Changes

  • Backend: Added chat_settings_location and default_chat_settings_open options to UISettings.
  • Frontend: Implemented ChatSettingsSidebar using the ResizablePanel pattern.
  • UI: Added a settings gear icon to the header (left of theme toggle) that toggles the sidebar.

Configuration

To enable the sidebar mode:

[UI]
chat_settings_location = "sidebar"
default_chat_settings_open = true
image

Summary by cubic

Adds an optional chat settings sidebar with a header toggle. When enabled, settings move from the message composer modal to a resizable right sidebar that can open by default.

  • New Features

    • Backend: Added UISettings options chat_settings_location ("message_composer" | "sidebar") and default_chat_settings_open; updated react-client config types.
    • Frontend: New ChatSettingsSidebar (resizable on desktop, Sheet on mobile).
    • Header: Added gear button to toggle the sidebar (shown only when settings exist).
    • Composer: Hides the settings modal trigger when sidebar mode is enabled.
  • Migration

    • To enable: UI.chat_settings_location = "sidebar".
    • Optional: UI.default_chat_settings_open = true to open the sidebar on load.

Written for commit 5bf65ee. Summary will update on new commits.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. backend Pertains to the Python backend. frontend Pertains to the frontend. labels Jan 17, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 7 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="frontend/src/components/header/index.tsx">

<violation number="1" location="frontend/src/components/header/index.tsx:100">
P2: Add an accessible label to the icon-only settings button so screen readers can announce it.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

@hayescode hayescode added this pull request to the merge queue Jan 18, 2026
Merged via the queue into main with commit fcb35f8 Jan 18, 2026
10 checks passed
@hayescode hayescode deleted the feat/settings_sidebar branch January 18, 2026 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Pertains to the Python backend. frontend Pertains to the frontend. size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants