IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-29 12:13:58 -06:00
Code Issues Packages Projects Releases Wiki Activity

FIX: closing the picker shouldn't propagate the pointer event (#16266)

Browse Source
This commit is contained in:
Joffrey JAFFEUX 2022-03-23 16:42:53 +01:00 committed by GitHub
parent cbaf7c949b
commit 4a39850aac
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
app/assets/javascripts/discourse/app/components/emoji-picker.js
View File

@ -153,9 +153,10 @@ export default Component.extend({
},
@action
onClose() {
onClose(event) {
event?.stopPropagation();
document.removeEventListener("click", this.handleOutsideClick);
this.onEmojiPickerClose && this.onEmojiPickerClose();
this.onEmojiPickerClose && this.onEmojiPickerClose(event);
},
diversityScales: computed("selectedDiversity", function () {
@ -221,7 +222,7 @@ export default Component.extend({
});
if (this.site.isMobileDevice) {
this.onClose();
this.onClose(event);
}
},
@ -236,7 +237,7 @@ export default Component.extend({
@action
keydown(event) {
if (event.code === "Escape") {
this.onClose();
this.onClose(event);
return false;
}
},
@ -334,7 +335,7 @@ export default Component.extend({
handleOutsideClick(event) {
const emojiPicker = document.querySelector(".emoji-picker");
if (emojiPicker && !emojiPicker.contains(event.target)) {
this.onClose();
this.onClose(event);
}
},
});