Merge pull request #9 from adampetrie/add-custom-scoping-to-shortcuts

Allow scoped to be a selector and bind mousetrap to that selector instead of the current node.
Skalar · Dec 17, 2015 · 3a18a1f · 3a18a1f
2 parents 5517b26 + acbac10
commit 3a18a1f
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/addon/create-mixin.js b/addon/create-mixin.js
@@ -40,7 +40,11 @@ export default function(bindEvent, unbindEvent) {
           if (actionObject.global === false) {
             mousetrap = new Mousetrap(document);
           } else if (actionObject.scoped) {
-            mousetrap = new Mousetrap(self.get('element'));
+            if (Ember.typeOf(actionObject.scoped) === 'boolean') {
+              mousetrap = new Mousetrap(self.get('element'));
+            } else if (Ember.typeOf(actionObject.scoped) === 'string') {
+              mousetrap = new Mousetrap(document.querySelector(actionObject.scoped));
+            }
           } else if (actionObject.targetElement) {
             mousetrap = new Mousetrap(actionObject.targetElement);
           }