ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-02-01 17:23:35 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix editors being disposed multiple times (#5594)

Browse Source
This commit is contained in:
Anthony Dresser
2019-05-23 14:08:37 -07:00
committed by GitHub
parent 0d37047c37
commit 34ca0e8671
1 changed files with 1 additions and 25 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
src/sql/workbench/parts/query/browser/queryEditor.ts
View File

@@ -100,8 +100,7 @@ export class QueryEditor extends BaseEditor {
parent.appendChild(splitviewContainer);
this.splitview = new SplitView(splitviewContainer, { orientation: Orientation.VERTICAL });
this._register(this.splitview);
this.splitview = this._register(new SplitView(splitviewContainer, { orientation: Orientation.VERTICAL }));
this._register(this.splitview.onDidSashReset(() => this.splitview.distributeViewSizes()));
this.textEditorContainer = DOM.$('.text-editor-container');
@@ -345,13 +344,6 @@ export class QueryEditor extends BaseEditor {
}
}
public dispose(): void {
this.textEditor.dispose();
this.resultsEditor.dispose();
this.splitview.dispose();
super.dispose();
}
public close(): void {
let queryInput: QueryInput = <QueryInput>this.input;
queryInput.sql.close();
@@ -490,20 +482,4 @@ export class QueryEditor extends BaseEditor {
public chart(dataId: { batchId: number, resultId: number }) {
this.resultsEditor.chart(dataId);
}
/**
* Sets the text selection for the SQL editor based on the given ISelectionData.
*/
private _setSelection(selection: ISelectionData): void {
let rangeConversion: IRange = {
startLineNumber: selection.startLine + 1,
startColumn: selection.startColumn + 1,
endLineNumber: selection.endLine + 1,
endColumn: selection.endColumn + 1
};
let editor = this.textEditor.getControl();
editor.revealRange(rangeConversion);
editor.setSelection(rangeConversion);
editor.focus();
}
}