refactor; remove console.log

ilios · Dec 12, 2024 · e748a4b · e748a4b
1 parent 3fa46c9
commit e748a4b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 18 deletions.
diff --git a/packages/ilios-common/addon/components/course/publication-menu.js b/packages/ilios-common/addon/components/course/publication-menu.js
@@ -76,6 +76,7 @@ export default class CoursePublicationMenuComponent extends Component {
       item?.parentElement.lastElementChild.focus();
     }
   }
+
   async handleArrowDown(item) {
     if (item.classList.value == 'toggle') {
       this.isOpen = true;
@@ -90,25 +91,21 @@ export default class CoursePublicationMenuComponent extends Component {
   }
 
   @action
-  async toggleMenu(event) {
-    console.log('course toggleMenu', event.target);
+  async toggleMenu() {
     this.isOpen = !this.isOpen;
 
     if (this.isOpen) {
       await this.focusFirstLink.perform();
     }
   }
   @action
-  keyUp(event) {
-    event.stopPropagation();
-    event.preventDefault();
-
-    switch (event.key) {
+  keyUp({ key, target }) {
+    switch (key) {
       case 'ArrowDown':
-        this.handleArrowDown(event.target);
+        this.handleArrowDown(target);
         break;
       case 'ArrowUp':
-        this.handleArrowUp(event.target);
+        this.handleArrowUp(target);
         break;
       case 'Escape':
       case 'Tab':

diff --git a/packages/ilios-common/addon/components/session/publication-menu.js b/packages/ilios-common/addon/components/session/publication-menu.js
@@ -80,6 +80,7 @@ export default class SessionPublicationMenuComponent extends Component {
       item?.parentElement.lastElementChild.focus();
     }
   }
+
   async handleArrowDown(item) {
     if (item.classList.value == 'toggle') {
       this.isOpen = true;
@@ -94,25 +95,21 @@ export default class SessionPublicationMenuComponent extends Component {
   }
 
   @action
-  async toggleMenu(event) {
-    console.log('session toggleMenu', event.target);
+  async toggleMenu() {
     this.isOpen = !this.isOpen;
 
     if (this.isOpen) {
       await this.focusFirstLink.perform();
     }
   }
   @action
-  keyUp(event) {
-    event.stopPropagation();
-    event.preventDefault();
-
-    switch (event.key) {
+  keyUp({ key, target }) {
+    switch (key) {
       case 'ArrowDown':
-        this.handleArrowDown(event.target);
+        this.handleArrowDown(target);
         break;
       case 'ArrowUp':
-        this.handleArrowUp(event.target);
+        this.handleArrowUp(target);
         break;
       case 'Escape':
       case 'Tab':