Merge pull request #82 from JsSucks/fullscreen-modals

Merge

client/src/ui/components/BdSettingsWrapper.vue

@@ -40,6 +40,7 @@
             hideSettings() { this.active = false },
             toggleSettings() { this.active = !this.active },
             keyupListener(e) {
+                if (document.getElementsByClassName('bd-backdrop').length) return;
                 if (this.active && e.which === 27) return this.hideSettings();
                 if (!e.metaKey && !e.ctrlKey || e.key !== 'b') return;
                 this.toggleSettings();

client/src/ui/components/common/Modal.vue

@@ -44,6 +44,19 @@
             return {
                 scrolled: false
             };
+        },
+        beforeMount() {
+            window.addEventListener('keyup', this.keyupListener);
+        },
+        destroyed() {
+            window.removeEventListener('keyup', this.keyupListener);
+        },
+        methods: {
+            keyupListener(e) {
+                if (e.which === 27) {
+                    this.close();
+                }
+            }
         }
     }
 </script>