Aligned entity-selector-popup button and dblclick behaviour
Fixes #3534
This commit is contained in:
parent
47887ec595
commit
736d6afb7d
1 changed files with 8 additions and 9 deletions
|
@ -16,7 +16,7 @@ class EntitySelectorPopup {
|
||||||
|
|
||||||
this.selectButton.addEventListener('click', this.onSelectButtonClick.bind(this));
|
this.selectButton.addEventListener('click', this.onSelectButtonClick.bind(this));
|
||||||
window.$events.listen('entity-select-change', this.onSelectionChange.bind(this));
|
window.$events.listen('entity-select-change', this.onSelectionChange.bind(this));
|
||||||
window.$events.listen('entity-select-confirm', this.onSelectionConfirm.bind(this));
|
window.$events.listen('entity-select-confirm', this.handleConfirmedSelection.bind(this));
|
||||||
}
|
}
|
||||||
|
|
||||||
show(callback) {
|
show(callback) {
|
||||||
|
@ -34,14 +34,7 @@ class EntitySelectorPopup {
|
||||||
}
|
}
|
||||||
|
|
||||||
onSelectButtonClick() {
|
onSelectButtonClick() {
|
||||||
this.hide();
|
this.handleConfirmedSelection(this.selection);
|
||||||
if (this.selection !== null && this.callback) this.callback(this.selection);
|
|
||||||
}
|
|
||||||
|
|
||||||
onSelectionConfirm(entity) {
|
|
||||||
this.hide();
|
|
||||||
this.getSelector().reset();
|
|
||||||
if (this.callback && entity) this.callback(entity);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
onSelectionChange(entity) {
|
onSelectionChange(entity) {
|
||||||
|
@ -52,6 +45,12 @@ class EntitySelectorPopup {
|
||||||
this.selectButton.removeAttribute('disabled');
|
this.selectButton.removeAttribute('disabled');
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
handleConfirmedSelection(entity) {
|
||||||
|
this.hide();
|
||||||
|
this.getSelector().reset();
|
||||||
|
if (this.callback && entity) this.callback(entity);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
export default EntitySelectorPopup;
|
export default EntitySelectorPopup;
|
Loading…
Reference in a new issue