From a07880d0aba04165f1d6931c2e79dd5232b5a514 Mon Sep 17 00:00:00 2001 From: ilhan orhan Date: Mon, 10 Jun 2019 12:48:39 +0300 Subject: [PATCH] fix(ui5-li): remove active state onmouseup (#525) --- packages/main/src/ListItem.js | 21 ++++++++++++++++++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/packages/main/src/ListItem.js b/packages/main/src/ListItem.js index c1e0331dbd93..ddf45dd75093 100644 --- a/packages/main/src/ListItem.js +++ b/packages/main/src/ListItem.js @@ -80,8 +80,26 @@ class ListItem extends ListItemBase { return [styles, ListItemBase.styles]; } + constructor() { + super(); + + this.deactivate = () => { + if (this._active) { + this._active = false; + } + }; + } + onBeforeRendering() {} + onEnterDOM() { + document.addEventListener("mouseup", this.deactivate); + } + + onExitDOM() { + document.removeEventListener("mouseup", this.deactivate); + } + onkeydown(event) { super.onkeydown(event); @@ -141,9 +159,6 @@ class ListItem extends ListItemBase { } } - deactivate() { - this._active = false; - } _onDelete(event) { this.fireEvent("_selectionRequested", { item: this, selected: event.selected });