Merge pull request #559 from vyneer/feat/dismiss-selected-event

Dismiss selected event by clicking it again
destinygg · Nov 16, 2024 · 6a03937 · 6a03937
2 parents 1ea42a5 + 8da4c7d
commit 6a03937
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/assets/chat/css/chat/event-bar/_event-bar-event.scss b/assets/chat/css/chat/event-bar/_event-bar-event.scss
@@ -12,6 +12,7 @@
   border-color: unset;
   border-width: 2px;
   margin: a.$gutter-sm;
+  user-select: none;
 
   .event-contents {
     display: flex;

diff --git a/assets/chat/js/event-bar/EventBar.js b/assets/chat/js/event-bar/EventBar.js
@@ -49,6 +49,13 @@ export default class ChatEventBar extends EventEmitter {
     this.events.push(event);
 
     event.element.addEventListener('click', () => {
+      if (
+        this.eventSelectUI.querySelector(`[data-uuid='${event.data.uuid}']`)
+      ) {
+        this.unselect();
+        return;
+      }
+
       this.select(event.selectedElement);
     });
     event.on('eventExpired', this.removeEvent.bind(this));
@@ -59,9 +66,6 @@ export default class ChatEventBar extends EventEmitter {
 
     this.eventBarUI.prepend(event.element);
 
-    // // Update chat window to fix the scroll position
-    // this.chat.mainwindow.update();
-    //
     event.startExpiry();
   }