fix to use elem clicked instead of querying from doc root

ilios · Dec 13, 2024 · 640173d · 640173d
1 parent d3fcc0c
commit 640173d
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/packages/ilios-common/addon/components/session/publication-menu.js b/packages/ilios-common/addon/components/session/publication-menu.js
@@ -68,9 +68,9 @@ export default class SessionPublicationMenuComponent extends Component {
     return 'notpublished';
   }
 
-  focusFirstLink = task(async () => {
+  focusFirstLink = task(async (item) => {
     await timeout(1);
-    document.querySelector('.publication-menu .menu button:first-of-type').focus();
+    item.querySelector('.menu button:first-of-type').focus();
   });
 
   handleArrowUp(item) {
@@ -84,22 +84,22 @@ export default class SessionPublicationMenuComponent extends Component {
   async handleArrowDown(item) {
     if (item.classList.value == 'toggle') {
       this.isOpen = true;
-      await this.focusFirstLink.perform();
+      await this.focusFirstLink.perform(item.parentElement);
     } else {
       if (item.nextElementSibling) {
         item.nextElementSibling.focus();
       } else {
-        await this.focusFirstLink.perform();
+        await this.focusFirstLink.perform(item.parentElement);
       }
     }
   }
 
   @action
-  async toggleMenu() {
+  async toggleMenu({ target }) {
     this.isOpen = !this.isOpen;
 
     if (this.isOpen) {
-      await this.focusFirstLink.perform();
+      await this.focusFirstLink.perform(target.parentElement.parentElement);
     }
   }
   @action
@@ -122,9 +122,9 @@ export default class SessionPublicationMenuComponent extends Component {
     return true;
   }
   @action
-  clearFocus() {
-    const buttons = document.querySelectorAll('.publication-menu .menu button');
-    buttons.forEach((el) => el.blur());
+  clearFocus({ target }) {
+    const menu = target.parentElement.parentElement;
+    menu.querySelectorAll('button').forEach((el) => el.blur());
   }
   @action
   close() {