Add visual separator to chat terminal button when scrollbar is present #274434
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.
When the terminal tabs list overflows and displays a scrollbar, the scrollbar ends directly adjacent to the chat terminal button with no visual separation, making it difficult to distinguish the scrollable area from the button.
Changes
terminalTabsChatEntry.ts: Added overflow detection in
update()method.monaco-scrollable-elementand caches referencescrollHeightvsclientHeightto detect scrollbar presencehas-scroll-separatorclass dynamicallyterminalTabbedView.ts: Pass tabs list container to chat entry via
setTabsListContainer()terminal.css: Added border-top rule for
has-scroll-separatorclass using--vscode-terminal-bordertheme variableThe separator appears only when overflow is detected and uses the existing theme border color for consistency.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.