-
Notifications
You must be signed in to change notification settings - Fork 37.6k
Align chat title container with centered interactive-session content #291564
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
…ctive-session Co-authored-by: benibenj <44439583+benibenj@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Aligns the chat view’s title/toolbar area with the centered .interactive-session content to avoid visual misalignment in wide chat views.
Changes:
- Adds a
max-widthconstraint intended to match the chat content width (950px) - Centers the affected container via auto margins
Comments suppressed due to low confidence (1)
src/vs/workbench/contrib/chat/browser/widgetHosts/viewPane/media/chatViewPane.css:19
margin: autosets auto margins on all four sides. Because.chat-controls-containeris a flex item (its parent is.chat-viewpanewhich can beflex-direction: columnorrow), this can interfere with stretching and can absorb free space on the main axis. If the intent is only horizontal centering, use left/right auto margins explicitly and setwidth: 100%so the container fills available space beforemax-widthclamps it.
max-width: 950px;
margin: auto;
| max-width: 950px; | ||
| margin: auto; |
Copilot
AI
Jan 29, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change applies max-width/centering to .chat-controls-container, which constrains the entire chat pane (including the widget) rather than just the title container described in the PR. In side-by-side sessions layout (.chat-viewpane.has-sessions-control.sessions-control-orientation-sidebyside switches to flex-direction: row), this will cap the chat area to 950px and center it, likely leaving unused space and misaligning with the sessions column. Consider keeping .chat-controls-container full-width and instead applying the max-width + horizontal centering to the title container (and/or an inner wrapper that contains the title + .interactive-session).
…nter setting" This reverts commit 70ead1b.
The chat title and toolbar were left-aligned while
.interactive-sessioncontent was centered withmax-width: 950px. This created visual misalignment when the chat view is wide.Applied matching constraints to
.chat-view-title-container:max-width: 950pxto match.interactive-sessionmargin-left: auto; margin-right: autofor centeringwidth: 100%to fill available space before max-width appliesBefore: Title left-aligned, content centered

After: Title and content aligned (per @benibenj's suggestion)

Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.