Move state check ahead to ensure stale docs are not made visible #29468

This commit is contained in:
Ramya Achutha Rao
2017-06-26 10:14:25 -07:00
parent 8ace8efaf9
commit 99ffa6afcf

View File

@@ -850,6 +850,10 @@ export class SuggestWidget implements IContentWidget, IDelegate<ICompletionItem>
this.editor.layoutContentWidget(this);
this.telemetryService.publicLog('suggestWidget:collapseDetails', this.editor.getTelemetryData());
} else {
if (this.state !== State.Open && this.state !== State.Details) {
return;
}
this.updateExpandDocsSetting(true);
this.showDetails();
this.telemetryService.publicLog('suggestWidget:expandDetails', this.editor.getTelemetryData());
@@ -858,13 +862,10 @@ export class SuggestWidget implements IContentWidget, IDelegate<ICompletionItem>
}
showDetails(): void {
if (this.state !== State.Open && this.state !== State.Details) {
return;
}
this.expandSideOrBelow();
show(this.details.element);
this.renderDetails();
this.details.render(this.list.getFocusedElements()[0]);
this.details.element.style.maxHeight = this.maxWidgetHeight + 'px';
// Reset margin-top that was set as Fix for #26416
@@ -980,14 +981,6 @@ export class SuggestWidget implements IContentWidget, IDelegate<ICompletionItem>
}
}
private renderDetails(): void {
if (this.state === State.Details || this.state === State.Open) {
this.details.render(this.list.getFocusedElements()[0]);
} else {
this.details.render(null);
}
}
// Heights
private get maxWidgetHeight(): number {