From b3bb6ebc6e8ee4be8adf986bf272f45b984c80b5 Mon Sep 17 00:00:00 2001 From: Matt Irvine Date: Fri, 3 Aug 2018 18:12:18 -0700 Subject: [PATCH] Use event element to determine whether to hide dropdown (#2145) --- src/sql/base/browser/ui/editableDropdown/dropdown.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/sql/base/browser/ui/editableDropdown/dropdown.ts b/src/sql/base/browser/ui/editableDropdown/dropdown.ts index 90365401bf..ec63b80506 100644 --- a/src/sql/base/browser/ui/editableDropdown/dropdown.ts +++ b/src/sql/base/browser/ui/editableDropdown/dropdown.ts @@ -233,8 +233,8 @@ export class Dropdown extends Disposable { this._layoutTree(); return { dispose: () => { } }; }, - onDOMEvent: (e, activeElement) => { - if (!DOM.isAncestor(activeElement, this.$el.getHTMLElement()) && !DOM.isAncestor(activeElement, this.$treeContainer.getHTMLElement())) { + onDOMEvent: e => { + if (!DOM.isAncestor(e.srcElement, this.$el.getHTMLElement()) && !DOM.isAncestor(e.srcElement, this.$treeContainer.getHTMLElement())) { this._input.validate(); this._onBlur.fire(); this._contextView.hide();