From e3307326ef3b4a976f41d134d013b6ff2c4a6eb6 Mon Sep 17 00:00:00 2001
From: Joshua Goins <josh@redstrate.com>
Date: Wed, 14 Jan 2026 20:51:11 -0500
Subject: [PATCH] Close the message menu after selecting a quick reaction

And also ensure the "select an emoji" menu doesn't close the message
menu after *not* choosing an emoji, so it acts more like a submenu.
---
 src/timeline/DelegateContextMenu.qml | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/timeline/DelegateContextMenu.qml b/src/timeline/DelegateContextMenu.qml
index b0cfb94c8..f8b90cdd6 100644
--- a/src/timeline/DelegateContextMenu.qml
+++ b/src/timeline/DelegateContextMenu.qml
@@ -132,14 +132,12 @@ KirigamiComponents.ConvergentContextMenu {
                                 root.room.toggleReaction(root.eventId, emoji);
                                 root.close();
                             });
-                            dialog.closed.connect(() => {
-                                root.close();
-                            });
                             dialog.open();
                             return;
                         }
 
                         root.room.toggleReaction(root.eventId, modelData);
+                        root.close();
                     }
                 }
             }