Align chat title container with centered interactive-session content#291564
Align chat title container with centered interactive-session content#291564
Conversation
…ctive-session Co-authored-by: benibenj <44439583+benibenj@users.noreply.github.com>
There was a problem hiding this comment.
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; |
There was a problem hiding this comment.
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.