Skip to content

Commit a07880d

Browse files
authored
fix(ui5-li): remove active state onmouseup (#525)
1 parent 0f3b8e4 commit a07880d

File tree

1 file changed

+18
-3
lines changed

1 file changed

+18
-3
lines changed

packages/main/src/ListItem.js

+18-3
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,26 @@ class ListItem extends ListItemBase {
8080
return [styles, ListItemBase.styles];
8181
}
8282

83+
constructor() {
84+
super();
85+
86+
this.deactivate = () => {
87+
if (this._active) {
88+
this._active = false;
89+
}
90+
};
91+
}
92+
8393
onBeforeRendering() {}
8494

95+
onEnterDOM() {
96+
document.addEventListener("mouseup", this.deactivate);
97+
}
98+
99+
onExitDOM() {
100+
document.removeEventListener("mouseup", this.deactivate);
101+
}
102+
85103
onkeydown(event) {
86104
super.onkeydown(event);
87105

@@ -141,9 +159,6 @@ class ListItem extends ListItemBase {
141159
}
142160
}
143161

144-
deactivate() {
145-
this._active = false;
146-
}
147162

148163
_onDelete(event) {
149164
this.fireEvent("_selectionRequested", { item: this, selected: event.selected });

0 commit comments

Comments
 (0)