From 83106c5b7f9e636eb937932cdabdbe053b126eb1 Mon Sep 17 00:00:00 2001
From: Daniel Imms <daimms@microsoft.com>
Date: Sat, 6 Oct 2018 11:16:39 -0700
Subject: Apply selection even when the items don't change

Fixes #59876
---
 src/vs/base/browser/ui/selectBox/selectBoxCustom.ts | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/src/vs/base/browser/ui/selectBox/selectBoxCustom.ts b/src/vs/base/browser/ui/selectBox/selectBoxCustom.ts
index 3d5b2f79cfa..fe455555e10 100644
--- a/src/vs/base/browser/ui/selectBox/selectBoxCustom.ts
+++ b/src/vs/base/browser/ui/selectBox/selectBoxCustom.ts
@@ -250,7 +250,6 @@ export class SelectBoxList implements ISelectBoxDelegate, IVirtualDelegate<ISele
 	}
 
 	public setOptions(options: string[], selected?: number, disabled?: number): void {
-
 		if (!this.options || !arrays.equals(this.options, options)) {
 			this.options = options;
 			this.selectElement.options.length = 0;
@@ -260,18 +259,16 @@ export class SelectBoxList implements ISelectBoxDelegate, IVirtualDelegate<ISele
 				this.selectElement.add(this.createOption(option, i, disabled === i++));
 			});
 
-			if (selected !== undefined) {
-				this.select(selected);
-				// Set current = selected since this is not necessarily a user exit
-				this._currentSelection = this.selected;
-			}
-
 			if (disabled !== undefined) {
 				this.disabledOptionIndex = disabled;
 			}
 		}
 
-
+		if (selected !== undefined) {
+			this.select(selected);
+			// Set current = selected since this is not necessarily a user exit
+			this._currentSelection = this.selected;
+		}
 	}
 
 
-- 
cgit v1.2.3