Refactor chat UI for improved layout and consistency#311357
Open
mrleemurray wants to merge 9 commits intomainfrom
Open
Refactor chat UI for improved layout and consistency#311357mrleemurray wants to merge 9 commits intomainfrom
mrleemurray wants to merge 9 commits intomainfrom
Conversation
…lbar Co-authored-by: Copilot <copilot@github.com>
…izing and overflow handling Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors the Sessions chat UI styling to improve alignment/spacing and make picker labels behave better under constrained widths, aiming for a more consistent layout between the input toolbar and the bottom-row controls.
Changes:
- Update the send button icon to use
Codicon.arrowUp. - Left-align/realign chat widget containers and tweak spacing/padding to improve visual consistency.
- Improve overflow handling and responsive behavior (ellipsis + container query) for picker labels/toolbars.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/vs/sessions/contrib/chat/browser/newChatInput.ts | Changes the send button icon to arrowUp. |
| src/vs/sessions/contrib/chat/browser/media/newChatInSession.css | Left-aligns the chat widget content and tweaks label spacing. |
| src/vs/sessions/contrib/chat/browser/media/chatWidget.css | Adjusts container alignment/spacing and adds responsive overflow/ellipsis behavior for bottom-row pickers. |
| src/vs/sessions/contrib/chat/browser/media/chatInput.css | Refines padding/spacing, adds min-width/overflow constraints for toolbars, and aligns icon sizing. |
Comments suppressed due to low confidence (1)
src/vs/sessions/contrib/chat/browser/media/chatInput.css:263
.sessions-chat-attach-buttonis a custom clickable element (created as adivwithrole="button"), but it doesn’t settouch-action: manipulation;. Without this, taps can incur the 300ms delay on touch/pointer-event platforms (notably iOS). Addtouch-action: manipulation;to this rule to keep touch behavior consistent with the other toolbar/picker triggers.
.sessions-chat-attach-button {
display: flex;
align-items: center;
justify-content: center;
width: 22px;
height: 22px;
flex-shrink: 0;
border-radius: 4px;
cursor: pointer;
color: var(--vscode-icon-foreground);
background: transparent;
border: none;
outline: none;
}
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributor
Screenshot ChangesBase: Changed (4) |
Co-authored-by: Copilot <copilot@github.com>
…ing in chat toolbar for improved layout consistency Co-authored-by: Copilot <copilot@github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates the chat input and widget CSS to improve layout consistency, responsiveness, and label handling, particularly in narrow viewports. It also changes the send button icon for better clarity. The changes focus on ensuring toolbar and picker elements remain usable and visually aligned as the UI resizes, with improved ellipsis and spacing for long labels.
Screen.Recording.2026-04-20.at.12.58.38.mov
UI and Layout Improvements:
.sessions-chat-editor,.sessions-chat-toolbar, and.new-chat-widget-containerto improve consistency and anchoring of elements, especially as the viewport narrows [1] [2] [3] [4].Responsiveness and Label Handling:
min-width: 0andoverflow: hiddento multiple flex containers and toolbar elements, allowing labels to ellipsize gracefully when space is tight [1] [2] [3].Toolbar and Picker Styling:
.sessions-chat-dropdown-labeland related classes for better overflow handling and alignment [1] [2].Icon and Visual Tweaks:
Codicon.sendtoCodicon.arrowUpfor improved clarity.Minor Enhancements:
flex-shrink: 0to certain elements to prevent unwanted shrinking [1] [2].These changes collectively make the chat input UI more robust, visually consistent, and user-friendly across different screen sizes.