Commit fe1f660
sessions: refine Open in VS Code titlebar styling (#313056)
* sessions: refine Open in VS Code titlebar styling
Align the sessions titlebar button more closely with the core Open in Agents affordance.
Add the grayscale-to-color icon treatment with reduced-motion handling, switch the hover chrome to the secondary button background, and tune the separator and spacing around the adjacent Run split button.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* sessions: address PR review feedback on titlebar styling
- Fix incorrect menu ID references in LAYOUT.md (OpenSubMenu/TitleBarRight -> TitleBarSessionMenu/TitleBarRightLayout) and update file references to openInVSCode.contribution.ts and openInVSCodeWidget.ts
- Use --vscode-button-secondaryHoverBackground (hover token) instead of --vscode-button-secondaryBackground (resting token) for correct hover contrast
- Strengthen CSS selector specificity for Run split button margin so it beats the conflicting margin-left: 0 rule in changesTitleBarWidget.css
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* sessions: fix run action titlebar spacing regression
Target the Run split button's action-item wrapper so the 5px margin applies on the same element used by the titlebar action bar layout and the adjacent Changes override.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---------
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>1 parent ffc280d commit fe1f660
4 files changed
Lines changed: 43 additions & 22 deletions
File tree
- src/vs/sessions
- browser/parts/media
- contrib/chat/browser
- media
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
124 | 124 | | |
125 | 125 | | |
126 | 126 | | |
127 | | - | |
| 127 | + | |
128 | 128 | | |
129 | | - | |
| 129 | + | |
| 130 | + | |
130 | 131 | | |
131 | 132 | | |
132 | 133 | | |
| |||
664 | 665 | | |
665 | 666 | | |
666 | 667 | | |
| 668 | + | |
667 | 669 | | |
668 | 670 | | |
669 | 671 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
63 | 63 | | |
64 | 64 | | |
65 | 65 | | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
66 | 74 | | |
67 | 75 | | |
68 | 76 | | |
| |||
Lines changed: 29 additions & 18 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | | - | |
| 12 | + | |
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
22 | 33 | | |
23 | 34 | | |
24 | 35 | | |
| |||
31 | 42 | | |
32 | 43 | | |
33 | 44 | | |
| 45 | + | |
34 | 46 | | |
35 | 47 | | |
36 | 48 | | |
| |||
42 | 54 | | |
43 | 55 | | |
44 | 56 | | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
45 | 72 | | |
46 | 73 | | |
47 | 74 | | |
| |||
65 | 92 | | |
66 | 93 | | |
67 | 94 | | |
68 | | - | |
| 95 | + | |
69 | 96 | | |
70 | 97 | | |
71 | 98 | | |
72 | | - | |
73 | | - | |
74 | | - | |
75 | | - | |
76 | | - | |
77 | | - | |
78 | | - | |
79 | | - | |
80 | | - | |
81 | | - | |
82 | | - | |
83 | | - | |
84 | | - | |
85 | | - | |
86 | | - | |
87 | | - | |
88 | 99 | | |
89 | 100 | | |
90 | 101 | | |
| |||
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
40 | 40 | | |
41 | 41 | | |
42 | 42 | | |
43 | | - | |
44 | | - | |
| 43 | + | |
| 44 | + | |
45 | 45 | | |
46 | 46 | | |
47 | 47 | | |
| |||
0 commit comments