Skip to content

Commit e3f908a

Browse files
alexdimaalexdima
committed
Fix layering: Move IEditorConstructionOptions to /browser/
1 parent 5ab5f96 commit e3f908a

8 files changed

Lines changed: 34 additions & 34 deletions

File tree

‎src/vs/editor/browser/config/configuration.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,11 @@ import * as platform from 'vs/base/common/platform';
1111
import { CharWidthRequest, CharWidthRequestType, readCharWidths } from 'vs/editor/browser/config/charWidthReader';
1212
import { ElementSizeObserver } from 'vs/editor/browser/config/elementSizeObserver';
1313
import { CommonEditorConfiguration, IEnvConfiguration } from 'vs/editor/common/config/commonEditorConfig';
14-
import { EditorOption, IEditorConstructionOptions, EditorFontLigatures } from 'vs/editor/common/config/editorOptions';
14+
import { EditorOption, EditorFontLigatures } from 'vs/editor/common/config/editorOptions';
1515
import { BareFontInfo, FontInfo } from 'vs/editor/common/config/fontInfo';
1616
import { IDimension } from 'vs/editor/common/editorCommon';
1717
import { IAccessibilityService, AccessibilitySupport } from 'vs/platform/accessibility/common/accessibility';
18+
import { IEditorConstructionOptions } from 'vs/editor/browser/editorBrowser';
1819

1920
class CSSBasedConfigurationCache {
2021

‎src/vs/editor/browser/editorBrowser.ts‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -336,6 +336,18 @@ export interface IEditorAriaOptions {
336336
role?: string;
337337
}
338338

339+
export interface IEditorConstructionOptions extends IEditorOptions {
340+
/**
341+
* The initial editor dimension (to avoid measuring the container).
342+
*/
343+
dimension?: editorCommon.IDimension;
344+
/**
345+
* Place overflow widgets inside an external DOM node.
346+
* Defaults to an internal DOM node.
347+
*/
348+
overflowWidgetsDomNode?: HTMLElement;
349+
}
350+
339351
/**
340352
* A rich code editor.
341353
*/

‎src/vs/editor/browser/widget/codeEditorWidget.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import { ICodeEditorService } from 'vs/editor/browser/services/codeEditorService
2121
import { ICommandDelegate } from 'vs/editor/browser/view/viewController';
2222
import { IContentWidgetData, IOverlayWidgetData, View } from 'vs/editor/browser/view/viewImpl';
2323
import { ViewUserInputEvents } from 'vs/editor/browser/view/viewUserInputEvents';
24-
import { ConfigurationChangedEvent, EditorLayoutInfo, IEditorOptions, EditorOption, IComputedEditorOptions, FindComputedEditorOptionValueById, IEditorConstructionOptions, filterValidationDecorations } from 'vs/editor/common/config/editorOptions';
24+
import { ConfigurationChangedEvent, EditorLayoutInfo, IEditorOptions, EditorOption, IComputedEditorOptions, FindComputedEditorOptionValueById, filterValidationDecorations } from 'vs/editor/common/config/editorOptions';
2525
import { Cursor } from 'vs/editor/common/controller/cursor';
2626
import { CursorColumns } from 'vs/editor/common/controller/cursorCommon';
2727
import { ICursorPositionChangedEvent, ICursorSelectionChangedEvent } from 'vs/editor/common/controller/cursorEvents';
@@ -238,7 +238,7 @@ export class CodeEditorWidget extends Disposable implements editorBrowser.ICodeE
238238

239239
constructor(
240240
domElement: HTMLElement,
241-
options: IEditorConstructionOptions,
241+
options: editorBrowser.IEditorConstructionOptions,
242242
codeEditorWidgetOptions: ICodeEditorWidgetOptions,
243243
@IInstantiationService instantiationService: IInstantiationService,
244244
@ICodeEditorService codeEditorService: ICodeEditorService,
@@ -328,7 +328,7 @@ export class CodeEditorWidget extends Disposable implements editorBrowser.ICodeE
328328
this._codeEditorService.addCodeEditor(this);
329329
}
330330

331-
protected _createConfiguration(options: IEditorConstructionOptions, accessibilityService: IAccessibilityService): editorCommon.IConfiguration {
331+
protected _createConfiguration(options: editorBrowser.IEditorConstructionOptions, accessibilityService: IAccessibilityService): editorCommon.IConfiguration {
332332
return new Configuration(this.isSimpleWidget, options, this._domElement, accessibilityService);
333333
}
334334

‎src/vs/editor/common/config/editorOptions.ts‎

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import { Constants } from 'vs/base/common/uint';
1111
import { USUAL_WORD_SEPARATORS } from 'vs/editor/common/model/wordHelper';
1212
import { AccessibilitySupport } from 'vs/platform/accessibility/common/accessibility';
1313
import { IConfigurationPropertySchema } from 'vs/platform/configuration/common/configurationRegistry';
14-
import { IDimension } from 'vs/editor/common/editorCommon';
1514
import { IJSONSchema } from 'vs/base/common/jsonSchema';
1615

1716
//#region typed options
@@ -600,18 +599,6 @@ export interface IEditorOptions {
600599
showDeprecated?: boolean;
601600
}
602601

603-
export interface IEditorConstructionOptions extends IEditorOptions {
604-
/**
605-
* The initial editor dimension (to avoid measuring the container).
606-
*/
607-
dimension?: IDimension;
608-
/**
609-
* Place overflow widgets inside an external DOM node.
610-
* Defaults to an internal DOM node.
611-
*/
612-
overflowWidgetsDomNode?: HTMLElement;
613-
}
614-
615602
/**
616603
* @internal
617604
* The width of the minimap gutter, in pixels.

‎src/vs/editor/standalone/browser/standaloneCodeEditor.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,11 @@
55

66
import * as aria from 'vs/base/browser/ui/aria/aria';
77
import { Disposable, IDisposable, toDisposable, DisposableStore } from 'vs/base/common/lifecycle';
8-
import { ICodeEditor, IDiffEditor } from 'vs/editor/browser/editorBrowser';
8+
import { ICodeEditor, IDiffEditor, IEditorConstructionOptions } from 'vs/editor/browser/editorBrowser';
99
import { ICodeEditorService } from 'vs/editor/browser/services/codeEditorService';
1010
import { CodeEditorWidget } from 'vs/editor/browser/widget/codeEditorWidget';
1111
import { DiffEditorWidget } from 'vs/editor/browser/widget/diffEditorWidget';
12-
import { IDiffEditorOptions, IEditorOptions, IEditorConstructionOptions } from 'vs/editor/common/config/editorOptions';
12+
import { IDiffEditorOptions, IEditorOptions } from 'vs/editor/common/config/editorOptions';
1313
import { InternalEditorAction } from 'vs/editor/common/editorAction';
1414
import { IModelChangedEvent } from 'vs/editor/common/editorCommon';
1515
import { ITextModel } from 'vs/editor/common/model';

‎src/vs/editor/test/browser/testCodeEditor.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
* Licensed under the MIT License. See License.txt in the project root for license information.
44
*--------------------------------------------------------------------------------------------*/
55

6-
import { ICodeEditor, IActiveCodeEditor } from 'vs/editor/browser/editorBrowser';
6+
import { ICodeEditor, IActiveCodeEditor, IEditorConstructionOptions } from 'vs/editor/browser/editorBrowser';
77
import { IEditorContributionCtor } from 'vs/editor/browser/editorExtensions';
88
import { ICodeEditorService } from 'vs/editor/browser/services/codeEditorService';
99
import { View } from 'vs/editor/browser/view/viewImpl';
@@ -34,7 +34,7 @@ export interface ITestCodeEditor extends IActiveCodeEditor {
3434
class TestCodeEditor extends CodeEditorWidget implements ICodeEditor {
3535

3636
//#region testing overrides
37-
protected _createConfiguration(options: editorOptions.IEditorConstructionOptions): IConfiguration {
37+
protected _createConfiguration(options: IEditorConstructionOptions): IConfiguration {
3838
return new TestConfiguration(options);
3939
}
4040
protected _createView(viewModel: ViewModel): [View, boolean] {

‎src/vs/monaco.d.ts‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -3138,18 +3138,6 @@ declare namespace monaco.editor {
31383138
showDeprecated?: boolean;
31393139
}
31403140

3141-
export interface IEditorConstructionOptions extends IEditorOptions {
3142-
/**
3143-
* The initial editor dimension (to avoid measuring the container).
3144-
*/
3145-
dimension?: IDimension;
3146-
/**
3147-
* Place overflow widgets inside an external DOM node.
3148-
* Defaults to an internal DOM node.
3149-
*/
3150-
overflowWidgetsDomNode?: HTMLElement;
3151-
}
3152-
31533141
/**
31543142
* Configuration options for the diff editor.
31553143
*/
@@ -4400,6 +4388,18 @@ declare namespace monaco.editor {
44004388
readonly mode: string | null;
44014389
}
44024390

4391+
export interface IEditorConstructionOptions extends IEditorOptions {
4392+
/**
4393+
* The initial editor dimension (to avoid measuring the container).
4394+
*/
4395+
dimension?: IDimension;
4396+
/**
4397+
* Place overflow widgets inside an external DOM node.
4398+
* Defaults to an internal DOM node.
4399+
*/
4400+
overflowWidgetsDomNode?: HTMLElement;
4401+
}
4402+
44034403
/**
44044404
* A rich code editor.
44054405
*/

‎src/vs/workbench/contrib/scm/browser/scmViewPane.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ import { toResource, SideBySideEditor } from 'vs/workbench/common/editor';
5050
import { SIDE_BAR_BACKGROUND, SIDE_BAR_BORDER, PANEL_BACKGROUND, PANEL_INPUT_BORDER } from 'vs/workbench/common/theme';
5151
import { CodeEditorWidget, ICodeEditorWidgetOptions } from 'vs/editor/browser/widget/codeEditorWidget';
5252
import { ITextModel } from 'vs/editor/common/model';
53-
import { IEditorConstructionOptions } from 'vs/editor/common/config/editorOptions';
53+
import { IEditorConstructionOptions } from 'vs/editor/browser/editorBrowser';
5454
import { getSimpleEditorOptions } from 'vs/workbench/contrib/codeEditor/browser/simpleEditorOptions';
5555
import { IModelService } from 'vs/editor/common/services/modelService';
5656
import { EditorExtensionsRegistry } from 'vs/editor/browser/editorExtensions';

0 commit comments

Comments
 (0)