Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
Left-positioned tab actions still allow the terminal shoulder inset to shrink the visible tab cap.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Adjusts connected-tab close-button spacing to keep tab geometry consistent across terminal and wrapped states.
Changes:
- Updates connected-tab action padding and positioning.
- Adds geometry regression coverage.
- Adds a single-tab visual fixture.
File summaries
| File | Description |
|---|---|
connectedEditorTabs.css |
Adjusts close-action spacing. |
multiEditorTabsControl.test.ts |
Tests connected-tab geometry states. |
editorTabBar.fixture.ts |
Adds a single-tab fixture. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 1
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add explicit close-action hover fixtures for single, multiple, and wrapped tab layouts. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
19a981b
|
Added explicit close-action hover fixtures for the critical terminal layouts: single tab, last tab in a multi-tab row, upper wrapped row, and bottom wrapped row. These show the actual hover background so its inset spacing can be visually compared. Also narrowed width reservation to stable logical terminals, avoiding the viewport-edge geometry feedback loop seen in browser/Electron CI. |
Ensure every close-action hover fixture uses a clean active editor so the captured state shows the close glyph instead of a dirty indicator. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Resolve the connected tab CSS conflict by retaining the close-action alignment fixes while adopting the newer external shoulder reservation from main.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Fixes the connected document tab close action spacing so terminal shoulders and wrapped pill insets reserve the same space around the action as ordinary connected tabs. This keeps the visible tab cap width stable when closing down to a single tab.
Adds a geometry regression test covering multiple, single, bottom-wrapped, and upper-wrapped states, plus a single-tab component fixture.
Validation:
npm run eslint -- src/vs/workbench/test/browser/parts/editor/multiEditorTabsControl.test.ts src/vs/workbench/test/browser/componentFixtures/editor/editorTabBar.fixture.tsnpm run stylelint -- src/vs/workbench/contrib/modernUI/browser/media/connectedEditorTabs.css