refactor: clean up console logs + add ability to close context menu via hitting the escape key

Author: sghoweri

packages/uikit-workshop/src/scripts/components/pl-nav/pl-nav.js

@@ -202,7 +202,7 @@ class Nav extends BaseComponent {
     this.previousActiveLinks = [];
     this.iframeElem = document.querySelector('pl-iframe');
 
-    Mousetrap.bind('esc', function(e) {
+    Mousetrap(this).bind('esc', () => {
       self.cleanupActiveNav();
     });
 
@@ -260,10 +260,6 @@ class Nav extends BaseComponent {
           window.innerWidth <= 670 ||
           (window.innerWidth >= 670 && self.layoutMode !== 'vertical')
         ) {
-          console.log(
-            window.innerWidth <= 670 ||
-              (window.innerWidth >= 670 && self.layoutMode !== 'vertical')
-          );
           self.cleanupActiveNav();
         }
       } catch (error) {

packages/uikit-workshop/src/scripts/components/pl-tools-menu/pl-tools-menu.js

@@ -1,5 +1,6 @@
 /* eslint-disable no-unused-vars */
 import { define, props } from 'skatejs';
+import Mousetrap from 'mousetrap';
 import { h } from 'preact';
 import { BaseComponent } from '../base-component.js';
 import { urlHandler, patternName } from '../../utils';
@@ -49,6 +50,10 @@ class ToolsMenu extends BaseLitComponent {
 
     window.addEventListener('message', this.receiveIframeMessage, false);
     document.addEventListener('click', this.handleExternalClicks);
+
+    Mousetrap(this).bind('esc', () => {
+      this.close();
+    });
   }
 
   disconnecting() {
@@ -114,7 +119,6 @@ class ToolsMenu extends BaseLitComponent {
 
     if (data.event !== undefined && data.event === 'patternLab.pageClick') {
       try {
-        console.log('patternLab.pageClick');
         self.isOpen = false;
       } catch (error) {
         console.log(error);