Remove cloneElementFont Fixes: bpasero comments (microsoft#68332)

cleidigh · bpasero · commit 08a4504e0793 · 2019-02-10T11:56:20.000+01:00
diff --git a/src/vs/base/browser/ui/selectBox/selectBoxCustom.ts b/src/vs/base/browser/ui/selectBox/selectBoxCustom.ts
@@ -428,8 +428,6 @@ export class SelectBoxList implements ISelectBoxDelegate, IListVirtualDelegate<I
 		this.createSelectList(this.selectDropDownContainer);
 		this.setOptionsList();
 
-		this.cloneElementFont(this.selectElement, this.selectDropDownContainer);
-
 		// This allows us to flip the position based on measurement
 		// Set drop-down position above/below from required height and margins
 		// If pre-layout cannot fit at least one option do not show drop-down
@@ -709,13 +707,6 @@ export class SelectBoxList implements ISelectBoxDelegate, IListVirtualDelegate<I
 		return elementWidth;
 	}
 
-	private cloneElementFont(source: HTMLElement, target: HTMLElement) {
-		const fontSize = window.getComputedStyle(source, null).getPropertyValue('font-size');
-		const fontFamily = window.getComputedStyle(source, null).getPropertyValue('font-family');
-		target.style.fontFamily = fontFamily;
-		target.style.fontSize = fontSize;
-	}
-
 	private createSelectList(parent: HTMLElement): void {
 
 		// If we have already constructive list on open, skip
