Only update visibilities when visible (#84125)

This commit is contained in:
Christof Marti 2019-11-12 14:40:07 +01:00
parent cd39e8e0a2
commit 57ddcfd792

View file

@ -702,11 +702,11 @@ class QuickPick<T extends IQuickPickItem> extends QuickInput implements IQuickPi
}
protected update() {
this.ui.setVisibilities(this.canSelectMany ? { title: !!this.title || !!this.step, checkAll: true, inputBox: true, visibleCount: true, count: true, ok: true, list: true, message: !!this.validationMessage } : { title: !!this.title || !!this.step, inputBox: true, visibleCount: true, list: true, message: !!this.validationMessage, customButton: this.customButton, ok: this.ok });
super.update();
if (!this.visible) {
return;
}
this.ui.setVisibilities(this.canSelectMany ? { title: !!this.title || !!this.step, checkAll: true, inputBox: true, visibleCount: true, count: true, ok: true, list: true, message: !!this.validationMessage } : { title: !!this.title || !!this.step, inputBox: true, visibleCount: true, list: true, message: !!this.validationMessage, customButton: this.customButton, ok: this.ok });
super.update();
if (this.ui.inputBox.value !== this.value) {
this.ui.inputBox.value = this.value;
}
@ -860,11 +860,11 @@ class InputBox extends QuickInput implements IInputBox {
}
protected update() {
this.ui.setVisibilities({ title: !!this.title || !!this.step, inputBox: true, message: true });
super.update();
if (!this.visible) {
return;
}
this.ui.setVisibilities({ title: !!this.title || !!this.step, inputBox: true, message: true });
super.update();
if (this.ui.inputBox.value !== this.value) {
this.ui.inputBox.value = this.value;
}