IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-25 18:55:32 -06:00
Code Issues Packages Projects Releases Wiki Activity

FIX: Non-Interfering Backdrop Clicks (#21916)

Browse Source 
Previously, there was an issue where closing the message actions menu on mobile would unintentionally trigger a click event on an element below it, such as a thread indicator or a reaction. With the recent fix, this problem has been resolved. Now, when you close the menu, it will no longer interfere with or activate any elements positioned underneath it.
This commit is contained in:
Joffrey JAFFEUX
2023-06-07 11:40:43 +02:00
committed by GitHub
parent ab1bb0cfa0
commit 330137e7e4
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
plugins/chat/assets/javascripts/discourse/components/chat-message-actions-mobile.hbs
View File

@@ -9,7 +9,7 @@
<div <div
role="button" role="button"
class="collapse-area" class="collapse-area"
{{on "touchstart" this.collapseMenu passive=true}} {{on "touchstart" this.collapseMenu}}
> >
</div> </div>

2
plugins/chat/assets/javascripts/discourse/components/chat-message-actions-mobile.js
View File

@@ -48,7 +48,7 @@ export default class ChatMessageActionsMobile extends Component {
@action @action
collapseMenu(event) { collapseMenu(event) {
event.stopPropagation(); event.preventDefault();
this.#onCloseMenu(); this.#onCloseMenu();
} }