remove additional appendChild

sbatten · sbatten · commit 7182cbab075d · 2020-08-07T07:44:50.000-07:00
fixes microsoft#104224
diff --git a/src/vs/base/browser/ui/menu/menu.ts b/src/vs/base/browser/ui/menu/menu.ts
@@ -853,8 +853,6 @@ class SubmenuMenuActionViewItem extends BaseMenuActionViewItem {
 				this.parentData.submenu.style(this.menuStyle);
 			}
 
-			this.element.appendChild(this.submenuContainer);
-
 			// layout submenu
 			const entryBox = this.element.getBoundingClientRect();
 			const entryBoxUpdated = {

Original file line number	Diff line number	Diff line change
`@@ -853,8 +853,6 @@ class SubmenuMenuActionViewItem extends BaseMenuActionViewItem {`
`853`	`853`	`this.parentData.submenu.style(this.menuStyle);`
`854`	`854`	`}`
`855`	`855`
`856`		`- this.element.appendChild(this.submenuContainer);`
`857`		`-`
`858`	`856`	`// layout submenu`
`859`	`857`	`const entryBox = this.element.getBoundingClientRect();`
`860`	`858`	`const entryBoxUpdated = {`