fix: user list elements not getting removed sometimes

destinygg · Jul 9, 2024 · 27ee426 · 27ee426
1 parent 19d966e
commit 27ee426
Showing 1 changed file with 14 additions and 10 deletions.
diff --git a/assets/chat/js/menus/ChatUserMenu.js b/assets/chat/js/menus/ChatUserMenu.js
@@ -165,23 +165,25 @@ export default class ChatUserMenu extends ChatMenu {
   }
 
   addAndRedraw(user) {
-    if (!this.hasElement(user)) {
+    if (!this.getElement(user)) {
       this.addElement(user, true);
       this.filter();
       this.redraw();
     }
   }
 
   removeAndRedraw(user) {
-    if (this.hasElement(user)) {
-      this.removeElement(user);
+    const el = this.getElement(user);
+    if (el) {
+      this.removeElement(el);
       this.redraw();
     }
   }
 
   replaceAndRedraw(user) {
-    if (this.hasElement(user)) {
-      this.removeElement(user);
+    const el = this.getElement(user);
+    if (el) {
+      this.removeElement(el);
       this.addElement(user, true);
       this.filter();
       this.redraw();
@@ -227,8 +229,9 @@ export default class ChatUserMenu extends ChatMenu {
     this.container.append(section);
   }
 
-  removeElement(user) {
-    this.container.find(`.user-entry[data-user-id="${user.id}"]`).remove();
+  /** @param {HTMLElement} element */
+  removeElement(element) {
+    element.remove();
     this.totalcount -= 1;
   }
 
@@ -264,9 +267,10 @@ export default class ChatUserMenu extends ChatMenu {
     this.totalcount += 1;
   }
 
-  hasElement(user) {
-    return (
-      this.container.find(`.user-entry[data-user-id="${user.id}"]`).length > 0
+  getElement(user) {
+    const section = this.sections.get(this.highestSection(user));
+    return section.users.querySelector(
+      `.user-entry[data-user-id="${user.id}"]`,
     );
   }