'enter' bug

.gitignore

@@ -0,0 +1,4 @@
+node_modules
+.idea
+.grunt
+bower_components

.travis.yml

@@ -0,0 +1,5 @@
+language: node_js
+node_js:
+    - "0.10"
+before_script:
+    - npm install -g grunt-cli

Gruntfile.js

@@ -0,0 +1,73 @@
+module.exports = function(grunt) {
+
+  // Configuration.
+  grunt.initConfig({
+    // verifies we have formatted our js and HTML according to our style conventions
+    jsbeautifier: {
+      files: ['Gruntfile.js', 'jquery.hotkeys.js', 'test/spec/**/*.js'],
+      options: {
+        mode: 'VERIFY_ONLY',
+        js: {
+          'indent_size': 2,
+          'indent_char': ' ',
+          'indent_level': 0,
+          'indent_with_tabs': false,
+          'preserve_newlines': true,
+          'max_preserve_newlines': 2,
+          'jslint_happy': false,
+          'brace_style': 'end-expand',
+          'indent_scripts ': 'keep',
+          'keep_array_indentation': true,
+          'keep_function_indentation': false,
+          'space_before_conditional': true,
+          'break_chained_methods': false,
+          'eval_code': false,
+          'unescape_strings': false,
+          'wrap_line_length': 0
+        }
+      }
+    },
+    jshint: {
+      options: {
+        "undef": true,
+        "unused": true
+      },
+      files: 'jquery.hotkeys.js'
+    },
+    jasmine: {
+      pivotal: {
+        options: {
+          vendor: ['jquery-1.4.2.js', 'jquery.hotkeys.js', 'test/lib/**.js'],
+          outfile: 'test/SpecRunner.html',
+          keepRunner: true,
+          specs: 'test/spec/*Spec.js'
+        }
+      }
+    },
+    watch: {
+      scripts: {
+        files: ['**/*.js'],
+        tasks: ['default'],
+        options: {
+          spawn: false,
+          interrupt: true,
+          debounceDelay: 1000
+        }
+      }
+    }
+  });
+
+  grunt.loadNpmTasks('grunt-contrib-watch');
+
+  // Beautify javascript
+  grunt.loadNpmTasks("grunt-jsbeautifier");
+
+  // Task loading
+  grunt.loadNpmTasks("grunt-contrib-jshint");
+
+  // Running tests
+  grunt.loadNpmTasks('grunt-contrib-jasmine');
+
+  // Task registration.
+  grunt.registerTask("default", ["jsbeautifier", "jshint", "jasmine"]);
+};

README.md

@@ -1,93 +1,115 @@
+# jQuery.Hotkeys [![Build Status](https://secure.travis-ci.org/jeresig/jquery.hotkeys.png)](http://travis-ci.org/jeresig/jquery.hotkeys)
+
 #About
-**jQuery.hotkeys** is a plug-in that lets you easily add and remove handlers for keyboard events anywhere in your code supporting almost any key combination.  
+**jQuery Hotkeys** is a plug-in that lets you easily add and remove handlers for keyboard events anywhere in your code supporting almost any key combination.
 
-It is based on a library [Shortcut.js](http://www.openjs.com/scripts/events/keyboard_shortcuts/shortcut.js) written by [Binny V A](http://www.openjs.com/).
+This plugin is based off of the plugin by Tzury Bar Yochay: [jQuery.hotkeys](https://github.com/tzuryby/jquery.hotkeys)
 
 The syntax is as follows:
-<pre>
-$(expression).bind(<types>,<options>, <handler>);
-$(expression).unbind(<types>,<options>, <handler>);
 
-$(document).bind('keydown', 'Ctrl+a', fn);
+```javascript
+$(expression).bind(types, keys, handler);
+$(expression).unbind(types, handler);
+
+$(document).bind('keydown', 'ctrl+a', fn);
 
 // e.g. replace '$' sign with 'EUR'
 $('input.foo').bind('keyup', '$', function(){
-    this.value = this.value.replace('$', 'EUR');
+  this.value = this.value.replace('$', 'EUR');
 });
+```
 
-$('div.foo').unbind('keydown', 'Ctrl+a', fn);
-</pre>
-## [Live Demo](http://jshotkeys.googlepages.com/test-static-01.html)
+Syntax when wanting to use jQuery's `on()`/`off` methods:
 
-## Types
-Supported types are `'keydown'`, `'keyup'` and `'keypress'`
+```javascript
+$(expression).on(types, null, keys, handler);
+$(expression).off(types, handler);
 
-## Options
-The options are `'combi'` i.e. the key combination, and `'disableInInput'` which allow your code not to be executed when the cursor is located inside an input ( `$(elem).is('input') || $(elem).is('textarea')` ).
+$(document).on('keydown', null, 'ctrl+a', fn);
 
-As you can see, the key combination can be passed as string or as an object. You may pass an object in case you wish to override the default option for `disableInInput` which is set to `false`:
-<pre>
-$(document).bind('keydown', {combi:'a', disableinInput: true}, fn);
-</pre>
-I.e. when cursor is within an input field, `'a'` will be inserted into the input field without interfering. 
+// e.g. replace '$' sign with 'EUR'
+$('input.foo').on('keyup', null, '$', function(){
+  this.value = this.value.replace('$', 'EUR');
+});     
+```
 
-If you want to use more than one modifiers (e.g. alt+ctrl+z) you should define them by an alphabetical order e.g. alt+ctrl+shift
+## Example
 
-Modifiers are case insensitive, i.e. 'Ctrl+a' 'ctrl+a'.
+[Example](https://rawgit.com/jeresig/jquery.hotkeys/master/test-static-01.html)
 
-## Handler
-In previous versions there was an option propagate which is removed now and implemented at the user code level.
+## Event Types
 
-When using jQuery, if an event handler returns false, jQuery will call `stopPropagation()` and `preventDefault()`
+Supported types are `'keydown'`, `'keyup'` and `'keypress'`
 
 ## jQuery Compatibility
-Tested with *jQuery 1.2.6*
 
-It known to be working with all the major browsers on all available platforms (Win/Mac/Linux)
+Works with jQuery 1.4.2 and newer.
+
+It is known to be working with all the major browsers on all available platforms (Win/Mac/Linux)
+
+ * IE 6/7/8+
+ * FF 1.5/2/3+
+ * Opera-9+
+ * Safari-3+
+ * Chrome-0.2+
+
+## Browserify Compatibility
+If you want to include this module in a Browserified project, just add it to node_modules and require it.
+```javascript
+require('jquery.javascript');
+```
+
+This will work if jQuery is global (ex. served from a CDN). If it's not, you need to [shim it](https://github.com/thlorenz/browserify-shim#a-expose-global-variables-via-global):
+```javascript
+{
+  "browserify-shim": {
+    "jquery": "global:jQuery"
+  }
+}
+```
 
- * IE 6/7/8
- * FF 1.5/2/3
- * Opera-9
- * Safari-3
- * Chrome-0.2
+## Notes
 
-## Features added in this version (0.7.x)
- * Implemented as $.fn - let you use `this`.
- * jQuery selectors are supported.
- * Extending `$.fn.bind` and `$.fn.unbind` so you get a single interface for binding events to handlers
-
-## Overriding jQuery
-The plugin wraps the following jQuery methods:
- * $.fn.bind
- * $.fn.unbind
- * $.find
+Modifiers are not case sensitive (`Ctrl` == `ctrl` == `cTRL`)
 
-Even though the plugin overrides these methods, the original methods will *always* be called.
+If you want to use more than one modifier (e.g. `alt+ctrl+z`) you should define them by an alphabetical order e.g. alt+ctrl+shift
 
-The plugin will add functionality only for the `keydown`, `keyup` and `keypress` event types. Any other types are passed untouched to the original `'bind()'` and `'unbind()'` methods.
+Hotkeys aren't tracked if you're inside of an input element (unless you explicitly bind the hotkey directly to the input). This helps to avoid conflict with normal user typing.
 
-Moreover, if you call `bind()` without passing the shortcut key combination e.g. `$(document).bind('keydown', fn)` only the original `'bind()'` method will be executed.
+You can use namespacing by adding a suffix to the event type (e.g. `keyup.toggle`)
 
-I also modified the `$.fn.find` method by adding a single line at the top of the function body. here is the code:
 
-<pre>
-    jQuery.fn.find = function( selector ) {
-        // the line I added
-        this.query=selector;
-        // call jQuery original find
-        return jQuery.fn.__find__.apply(this, arguments);
-    };
-</pre>
+## Hotkeys within inputs
 
-You can read about this at [jQuery's User Group](http://groups.google.com/group/jquery-en/browse_thread/thread/18f9825e8d22f18d)
+Hotkeys aren't tracked if the user is focused within an input element or any element that has `contenteditable="true"` unless you bind the hotkey directly to the element. This helps to avoid conflict with normal user typing.
+If you don't want this, you can change the booleans of the following to suit:
 
-###Notes
+ * `jQuery.hotkeys.options.filterInputAcceptingElements`
+ * `jQuery.hotkeys.options.filterContentEditable`
+ * `jQuery.hotkeys.options.filterTextInputs` (deprecated, will be removed in a later version)
+
+### Meta and Hyper Keys
+Meta and hyper keys don't register on `keyup` in any browser tested.
+
+#### Chrome 33.0.1750.117
+Meta key registers on `keydown` event.
+Hyper key registers on `keydown` event.
+
+#### Firefox 27.0.1 and Safari 7.0.1
+Meta key registers on `keydown` and `keypress` events.
+Hyper key registers on `keydown` and `keypress` events.
+
+#### Opera 19.0
+Meta key doesn't register at all :(
+Hyper key registers on `keydown` and `keypress` events.
+
+#### TL;DR
+Bind to `keydown` event for meta and hyper keys, but meta key does not work in Opera ;)
+
+### Addendum
 
 Firefox is the most liberal one in the manner of letting you capture all short-cuts even those that are built-in in the browser such as `Ctrl-t` for new tab, or `Ctrl-a` for selecting all text. You can always bubble them up to the browser by returning `true` in your handler.
 
 Others, (IE) either let you handle built-in short-cuts, but will add their functionality after your code has executed. Or (Opera/Safari) will *not* pass those events to the DOM at all.
 
 *So, if you bind `Ctrl-Q` or `Alt-F4` and your Safari/Opera window is closed don't be surprised.*
-
-
-###Current Version is: beta 0.7

bower.json

@@ -0,0 +1,21 @@
+{
+  "name": "jquery.hotkeys",
+  "description": "jQuery Hotkeys lets you watch for keyboard events anywhere in your code supporting almost any key combination.",
+  "homepage": "http://github.com/jeresig/jquery.hotkeys",
+  "repository": "git://github.com/jeresig/jquery.hotkeys.git",
+  "keywords": ["jquery", "hotkeys", "keyboard", "events", "key", "bindings"],
+  "authors": ["John Resig <[email protected]> (http://ejohn.org)"],
+  "main": "jquery.hotkeys.js",
+  "ignore": [
+    "jquery-*",
+    "test*",
+    "*.yml",
+    ".gitignore",
+    "Gruntfile.js",
+    "*.json"
+  ],
+  "dependencies": {
+    "jquery": ">= 1.4.2"
+  },
+  "license": ["MIT", "GPL-2.0"]
+}

hotkeys.jquery.json

@@ -0,0 +1,36 @@
+{
+  "name": "hotkeys",
+  "title": "jQuery Hotkeys",
+  "description": "jQuery Hotkeys lets you watch for keyboard events anywhere in your code supporting almost any key combination.",
+  "keywords": [
+    "keyboard",
+    "events"
+  ],
+  "version": "0.2.0",
+  "author": {
+    "name": "John Resig",
+    "url": "https://ejohn.org/"
+  },
+  "maintainers": [
+    {
+      "name": "John Resig",
+      "email": "[email protected]",
+      "url": "http://ejohn.org/"
+    }
+  ],
+  "licenses": [
+    {
+      "type": "MIT",
+      "url": "http://opensource.org/licenses/MIT"
+    },
+    {
+      "type": "GPLv2",
+      "url": "http://www.gnu.org/licenses/gpl-2.0.html"
+    }
+  ],
+  "homepage": "https://github.com/jeresig/jquery.hotkeys",
+  "docs": "https://github.com/jeresig/jquery.hotkeys",
+  "dependencies": {
+    "jquery": ">=1.4.2"
+  }
+}