fixes microsoft#55439

mustafa-95 · Aug 2, 2018 · 49bce6d · 49bce6d
1 parent d91fdfd
commit 49bce6d
Showing 1 changed file with 18 additions and 2 deletions.
diff --git a/src/vs/base/browser/ui/menu/menu.ts b/src/vs/base/browser/ui/menu/menu.ts
@@ -241,6 +241,8 @@ class MenuActionItem extends BaseActionItem {
 class SubmenuActionItem extends MenuActionItem {
 	private mysubmenu: Menu;
 	private submenuContainer: Builder;
+	private mouseOver: boolean;
+	private showScheduler: RunOnceScheduler;
 	private hideScheduler: RunOnceScheduler;
 
 	constructor(
@@ -251,6 +253,13 @@ class SubmenuActionItem extends MenuActionItem {
 	) {
 		super(action, action, { label: true, isMenu: true });
 
+		this.showScheduler = new RunOnceScheduler(() => {
+			if (this.mouseOver) {
+				this.cleanupExistingSubmenu(false);
+				this.createSubmenu(false);
+			}
+		}, 250);
+
 		this.hideScheduler = new RunOnceScheduler(() => {
 			if ((!isAncestor(document.activeElement, this.builder.getHTMLElement()) && this.parentData.submenu === this.mysubmenu)) {
 				this.parentData.parent.focus(false);
@@ -283,8 +292,15 @@ class SubmenuActionItem extends MenuActionItem {
 		});
 
 		$(this.builder).on(EventType.MOUSE_OVER, (e) => {
-			this.cleanupExistingSubmenu(false);
-			this.createSubmenu(false);
+			if (!this.mouseOver) {
+				this.mouseOver = true;
+
+				this.showScheduler.schedule();
+			}
+		});
+
+		$(this.builder).on(EventType.MOUSE_LEAVE, (e) => {
+			this.mouseOver = false;
 		});
 
 		$(this.builder).on(EventType.FOCUS_OUT, (e) => {