Skip to content

Commit 085c409

Browse files
authored
Merge pull request microsoft#165575 from QaidVoid/vpln
Reduce registerThemingParticipant
2 parents aadbe18 + 99d7a55 commit 085c409

File tree

4 files changed

+22
-41
lines changed

4 files changed

+22
-41
lines changed

src/vs/editor/browser/viewParts/lineNumbers/lineNumbers.css

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,3 +23,11 @@
2323
.monaco-editor .margin-view-overlays .line-numbers.lh-odd {
2424
margin-top: 1px;
2525
}
26+
27+
.monaco-editor .line-numbers {
28+
color: var(--vscode-editorLineNumber-foreground);
29+
}
30+
31+
.monaco-editor .line-numbers.active-line-number {
32+
color: var(--vscode-editorLineNumber-activeForeground);
33+
}

src/vs/editor/browser/viewParts/lineNumbers/lineNumbers.ts

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,9 @@ import * as platform from 'vs/base/common/platform';
88
import { DynamicViewOverlay } from 'vs/editor/browser/view/dynamicViewOverlay';
99
import { RenderLineNumbersType, EditorOption } from 'vs/editor/common/config/editorOptions';
1010
import { Position } from 'vs/editor/common/core/position';
11-
import { editorActiveLineNumber, editorLineNumbers } from 'vs/editor/common/core/editorColorRegistry';
1211
import { RenderingContext } from 'vs/editor/browser/view/renderingContext';
1312
import { ViewContext } from 'vs/editor/common/viewModel/viewContext';
1413
import * as viewEvents from 'vs/editor/common/viewEvents';
15-
import { registerThemingParticipant } from 'vs/platform/theme/common/themeService';
1614

1715
export class LineNumbersOverlay extends DynamicViewOverlay {
1816

@@ -192,16 +190,3 @@ export class LineNumbersOverlay extends DynamicViewOverlay {
192190
return this._renderResult[lineIndex];
193191
}
194192
}
195-
196-
// theming
197-
198-
registerThemingParticipant((theme, collector) => {
199-
const lineNumbers = theme.getColor(editorLineNumbers);
200-
if (lineNumbers) {
201-
collector.addRule(`.monaco-editor .line-numbers { color: ${lineNumbers}; }`);
202-
}
203-
const activeLineNumber = theme.getColor(editorActiveLineNumber);
204-
if (activeLineNumber) {
205-
collector.addRule(`.monaco-editor .line-numbers.active-line-number { color: ${activeLineNumber}; }`);
206-
}
207-
});
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
/*---------------------------------------------------------------------------------------------
2+
* Copyright (c) Microsoft Corporation. All rights reserved.
3+
* Licensed under the MIT License. See License.txt in the project root for license information.
4+
*--------------------------------------------------------------------------------------------*/
5+
6+
.monaco-editor .findOptionsWidget {
7+
background-color: var(--vscode-editorWidget-background);
8+
color: var(--vscode-editorWidget-foreground);
9+
box-shadow: 0 0 8px 2px var(--vscode-widget-shadow);
10+
border: 2px solid var(--vscode-contrastBorder);
11+
}

src/vs/editor/contrib/find/browser/findOptionsWidget.ts

Lines changed: 3 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,16 @@
44
*--------------------------------------------------------------------------------------------*/
55

66
import * as dom from 'vs/base/browser/dom';
7+
import 'vs/css!./findOptionsWidget';
78
import { CaseSensitiveToggle, RegexToggle, WholeWordsToggle } from 'vs/base/browser/ui/findinput/findInputToggles';
89
import { Widget } from 'vs/base/browser/ui/widget';
910
import { RunOnceScheduler } from 'vs/base/common/async';
1011
import { ICodeEditor, IOverlayWidget, IOverlayWidgetPosition, OverlayWidgetPositionPreference } from 'vs/editor/browser/editorBrowser';
1112
import { FIND_IDS } from 'vs/editor/contrib/find/browser/findModel';
1213
import { FindReplaceState } from 'vs/editor/contrib/find/browser/findState';
1314
import { IKeybindingService } from 'vs/platform/keybinding/common/keybinding';
14-
import { contrastBorder, editorWidgetBackground, editorWidgetForeground, inputActiveOptionBackground, inputActiveOptionBorder, inputActiveOptionForeground, widgetShadow } from 'vs/platform/theme/common/colorRegistry';
15-
import { IColorTheme, IThemeService, registerThemingParticipant } from 'vs/platform/theme/common/themeService';
15+
import { inputActiveOptionBackground, inputActiveOptionBorder, inputActiveOptionForeground } from 'vs/platform/theme/common/colorRegistry';
16+
import { IColorTheme, IThemeService } from 'vs/platform/theme/common/themeService';
1617

1718
export class FindOptionsWidget extends Widget implements IOverlayWidget {
1819

@@ -198,27 +199,3 @@ export class FindOptionsWidget extends Widget implements IOverlayWidget {
198199
this.regex.style(inputStyles);
199200
}
200201
}
201-
202-
203-
registerThemingParticipant((theme, collector) => {
204-
const widgetBackground = theme.getColor(editorWidgetBackground);
205-
if (widgetBackground) {
206-
collector.addRule(`.monaco-editor .findOptionsWidget { background-color: ${widgetBackground}; }`);
207-
}
208-
209-
const widgetForeground = theme.getColor(editorWidgetForeground);
210-
if (widgetForeground) {
211-
collector.addRule(`.monaco-editor .findOptionsWidget { color: ${widgetForeground}; }`);
212-
}
213-
214-
215-
const widgetShadowColor = theme.getColor(widgetShadow);
216-
if (widgetShadowColor) {
217-
collector.addRule(`.monaco-editor .findOptionsWidget { box-shadow: 0 0 8px 2px ${widgetShadowColor}; }`);
218-
}
219-
220-
const hcBorder = theme.getColor(contrastBorder);
221-
if (hcBorder) {
222-
collector.addRule(`.monaco-editor .findOptionsWidget { border: 2px solid ${hcBorder}; }`);
223-
}
224-
});

0 commit comments

Comments
 (0)