Context menu bug in macOS fixed (microsoft#350)

* context menu now doesnt show in origin on mac * fixed context menu action being called twice in macOS
kod3r · Dec 18, 2017 · f6497a9 · f6497a9
1 parent 67aac4e
commit f6497a9
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/sql/parts/connection/connectionDialog/recentConnectionTreeController.ts b/src/sql/parts/connection/connectionDialog/recentConnectionTreeController.ts
@@ -89,7 +89,7 @@ export class RecentConnectionTreeController extends DefaultController {
 
 	protected onRightClick(tree: ITree, element: any, eventish: ICancelableEvent, origin: string = 'mouse'): boolean {
 		this.clickcb(element, eventish, origin);
-		this.onContextMenu(tree, element, event);
+		this.showContextMenu(tree, element, event);
 		return true;
 	}
 
@@ -113,7 +113,7 @@ export class RecentConnectionTreeController extends DefaultController {
 		return super.onKeyDown(tree, event);
 	}
 
-	public onContextMenu(tree: ITree, element: any, event: any): boolean {
+	public showContextMenu(tree: ITree, element: any, event: any): boolean {
 		var actionContext: any;
 
 		if (element instanceof ConnectionProfile) {
@@ -136,7 +136,6 @@ export class RecentConnectionTreeController extends DefaultController {
 			},
 			getActionsContext: () => (actionContext)
 		});
-
 		return true;
 	}
 }