Skip to content

Commit d6ae90f

Browse files
fix: Fix regression related to ability to show banner message inside the editor
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com> Assisted-by: Cursor AI
1 parent 244b360 commit d6ae90f

13 files changed

Lines changed: 75 additions & 10 deletions

File tree

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
[
2+
{
3+
"from": "\t\t\"--vscode-agentStatusIndicator-background\",\n\t\t\"--vscode-badge-background\",",
4+
"by": "\t\t\"--vscode-agentStatusIndicator-background\",\n\t\t\"--vscode-agentStatusIndicator-labelForeground\",\n\t\t\"--vscode-badge-background\","
5+
}
6+
]
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
[
2+
{
3+
"from": "\tcolor: var(--vscode-titleBar-activeForeground);\n\tbackground-color: var(--vscode-titleBar-activeBackground);\n\tborder-bottom: 1px solid var(--vscode-titleBar-border, transparent);",
4+
"by": "\tcolor: var(--vscode-editorGroupHeader-tabsForeground, var(--vscode-foreground));\n\tbackground-color: var(--vscode-editorGroupHeader-tabsBackground);\n\tborder-bottom: 1px solid var(--vscode-editorGroupHeader-tabsBorder, transparent);"
5+
},
6+
{
7+
"from": "\t\tcolor: var(--vscode-titleBar-activeForeground);\n\t\toverflow: hidden;",
8+
"by": "\t\tcolor: inherit;\n\t\toverflow: hidden;"
9+
},
10+
{
11+
"from": "\t\t\tbackground-color: var(--vscode-titleBar-activeForeground);\n\t\t\topacity: 0.3;",
12+
"by": "\t\t\tbackground-color: currentColor;\n\t\t\topacity: 0.3;"
13+
}
14+
]

.rebase/replace/code/src/vs/workbench/browser/parts/titlebar/windowTitle.ts.json

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@
55
},
66
{
77
"from": "\tisCustomTitleFormat(): boolean {",
8-
"by": "\tgetHeader(): string | undefined{\n\t\tconst header = this.configurationService.inspect<string>(WindowSettingNames.header);\n\t\treturn header.value;\n\t}\n\n\tisCustomTitleFormat(): boolean {"
8+
"by": "\tgetHeader(): string | undefined{\n\t\treturn this.configurationService.getValue<string>(WindowSettingNames.header);\n\t}\n\n\tisCustomTitleFormat(): boolean {"
9+
},
10+
{
11+
"from": "\t\tif (affectsTitleConfiguration || event.affectsConfiguration(WindowSettingNames.titleSeparator)) {\n\t\t\tthis.titleUpdater.schedule();\n\t\t}\n\t}",
12+
"by": "\t\tif (affectsTitleConfiguration || event.affectsConfiguration(WindowSettingNames.titleSeparator)) {\n\t\t\tthis.titleUpdater.schedule();\n\t\t}\n\n\t\tif (event.affectsConfiguration(WindowSettingNames.header)) {\n\t\t\tthis.onDidChangeEmitter.fire();\n\t\t}\n\t}"
913
}
1014
]
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
[
2+
{
3+
"from": ");\nexport const COMMAND_CENTER_ACTIVEFOREGROUND = registerColor(\n\t'commandCenter.activeForeground',",
4+
"by": ");\nexport const AGENT_STATUS_INDICATOR_LABEL_FOREGROUND = registerColor(\n\t'agentStatusIndicator.labelForeground',\n\ttransparent(COMMAND_CENTER_FOREGROUND, 0.6),\n\tlocalize('agentStatusIndicator-labelForeground', \"Foreground color of the agent status indicator label in the titlebar. Defaults to a muted version of the command center foreground.\"),\n\tfalse\n);\nexport const COMMAND_CENTER_ACTIVEFOREGROUND = registerColor(\n\t'commandCenter.activeForeground',"
5+
}
6+
]
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
[
2+
{
3+
"from": "\t\tlet label = this._windowTitle.workspaceName;\n\t\tif (this._windowTitle.isCustomTitleFormat()) {",
4+
"by": "\t\tlet label = this._windowTitle.workspaceName;\n\t\tconst header = this._windowTitle.getHeader();\n\t\tif (header) {\n\t\t\tlabel = header;\n\t\t} else if (this._windowTitle.isCustomTitleFormat()) {"
5+
}
6+
]
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
[
2+
{
3+
"from": "/* Label - styled as placeholder text */\n.agent-status-label {\n\tflex: 1;\n\ttext-align: center;\n\twhite-space: nowrap;\n\toverflow: hidden;\n\ttext-overflow: ellipsis;\n\topacity: 0.6;\n}",
4+
"by": "/* Label - styled as placeholder text */\n.agent-status-label {\n\tflex: 1;\n\ttext-align: center;\n\twhite-space: nowrap;\n\toverflow: hidden;\n\ttext-overflow: ellipsis;\n\tcolor: var(--vscode-agentStatusIndicator-labelForeground);\n}"
5+
}
6+
]

code/build/lib/stylelint/vscode-known-variables.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
"--vscode-agentSessionSelectedBadge-border",
2727
"--vscode-agentSessionSelectedUnfocusedBadge-border",
2828
"--vscode-agentStatusIndicator-background",
29+
"--vscode-agentStatusIndicator-labelForeground",
2930
"--vscode-badge-background",
3031
"--vscode-badge-foreground",
3132
"--vscode-banner-background",

code/src/vs/workbench/browser/parts/editor/media/modalEditorPart.css

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -90,9 +90,9 @@
9090
height: 32px;
9191
min-height: 32px;
9292
padding: 0 8px 0 10px;
93-
color: var(--vscode-titleBar-activeForeground);
94-
background-color: var(--vscode-titleBar-activeBackground);
95-
border-bottom: 1px solid var(--vscode-titleBar-border, transparent);
93+
color: var(--vscode-editorGroupHeader-tabsForeground, var(--vscode-foreground));
94+
background-color: var(--vscode-editorGroupHeader-tabsBackground);
95+
border-bottom: 1px solid var(--vscode-editorGroupHeader-tabsBorder, transparent);
9696

9797
/* Modal Editor Sidebar Toggle */
9898
.modal-editor-sidebar-toggle {
@@ -107,7 +107,7 @@
107107
grid-column: 2;
108108
font-size: 12px;
109109
font-weight: 500;
110-
color: var(--vscode-titleBar-activeForeground);
110+
color: inherit;
111111
overflow: hidden;
112112

113113
.monaco-icon-label {
@@ -185,7 +185,7 @@
185185
width: 1px;
186186
height: 16px;
187187
margin: 0 4px;
188-
background-color: var(--vscode-titleBar-activeForeground);
188+
background-color: currentColor;
189189
opacity: 0.3;
190190
}
191191
}

code/src/vs/workbench/browser/parts/titlebar/windowTitle.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,10 @@ export class WindowTitle extends Disposable {
140140
if (affectsTitleConfiguration || event.affectsConfiguration(WindowSettingNames.titleSeparator)) {
141141
this.titleUpdater.schedule();
142142
}
143+
144+
if (event.affectsConfiguration(WindowSettingNames.header)) {
145+
this.onDidChangeEmitter.fire();
146+
}
143147
}
144148

145149
private checkTitleVariables(): void {
@@ -409,8 +413,7 @@ export class WindowTitle extends Disposable {
409413
}
410414

411415
getHeader(): string | undefined{
412-
const header = this.configurationService.inspect<string>(WindowSettingNames.header);
413-
return header.value;
416+
return this.configurationService.getValue<string>(WindowSettingNames.header);
414417
}
415418

416419
isCustomTitleFormat(): boolean {

code/src/vs/workbench/common/theme.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -717,6 +717,12 @@ export const COMMAND_CENTER_FOREGROUND = registerColor(
717717
localize('commandCenter-foreground', "Foreground color of the command center"),
718718
false
719719
);
720+
export const AGENT_STATUS_INDICATOR_LABEL_FOREGROUND = registerColor(
721+
'agentStatusIndicator.labelForeground',
722+
transparent(COMMAND_CENTER_FOREGROUND, 0.6),
723+
localize('agentStatusIndicator-labelForeground', "Foreground color of the agent status indicator label in the titlebar. Defaults to a muted version of the command center foreground."),
724+
false
725+
);
720726
export const COMMAND_CENTER_ACTIVEFOREGROUND = registerColor(
721727
'commandCenter.activeForeground',
722728
MENUBAR_SELECTION_FOREGROUND,

0 commit comments

Comments
 (0)