Skip to content

Added the option to have multi lists active at the same time, a method to replace the active list and return it #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ricardovf wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Added the option to have multi lists active at the same time, a method to replace the active list and return it #3

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
62c8346
Added the ability to make more then one list active at the same time
Jul 11, 2012
f291ae0
Corrected little bug when we add a shortcut but does not start the list
Jul 11, 2012
2a22ad0
Added the replace option to the start method and a method to return t…
Jul 11, 2012
44dbb36
Removed indexOf (Internet Explorer sucks!)
Jul 11, 2012
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
83 changes: 66 additions & 17 deletions jquery.shortcuts.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@
/** Hash for shortcut lists */
var lists = {};

/** Active shortcut list */
var active;
/** Active shortcut lists */
var active = {};

/** Hash for storing which keys are pressed at the moment. Key - ASCII key code (e.which), value - true/false. */
var pressed = {};
Expand Down Expand Up @@ -101,8 +101,6 @@
};

var run = function(type, e) {
if (!active) { return; }

var maskObj = {
ctrl: e.ctrlKey,
alt: e.altKey,
Expand All @@ -111,7 +109,14 @@
};

var key = getKey(type, maskObj);
var shortcuts = active[key]; // Get shortcuts from the active list.

var shortcuts;

$.each(active, function(list, list_activated) {
if (list_activated === true && lists[list] && lists[list][key]) {
shortcuts = lists[list][key];
}
});

if (!shortcuts) { return; }

Expand All @@ -125,7 +130,7 @@
e.preventDefault();
isPrevented = true;
}
shortcut.handler(e); // Run the shortcut's handler.
shortcut.handler(shortcut, e); // Run the shortcut's handler.
}
});
};
Expand All @@ -134,11 +139,23 @@

/**
* Start reacting to shortcuts in the specified list.
* @param {String} [list] List name
* @param {String|Array} [list] List name or array of names
* @param {Boolean} [list] Should we replace the list of active lists?
*/
$.Shortcuts.start = function(list) {
list = list || 'default';
active = lists[list]; // Set the list as active.
$.Shortcuts.start = function(list, replace) {
if (list === undefined)
list = 'default';

if (typeof(list) === 'string')
list = [list]

// Set the lists as active
$.each(active, function(k, v) {
if ($.inArray(k, list) !== -1)
active[k] = true;
else
active[k] = (replace === true ? false : active[k]);
});

if (isStarted) { return; } // We are going to attach event handlers only once, the first time this method is called.

Expand All @@ -164,15 +181,44 @@
return this;
};

/**
* Stop reacting to shortcuts (unbind event handlers).
*/
$.Shortcuts.stop = function() {
$(document).unbind('keypress.shortcuts keydown.shortcuts keyup.shortcuts');
isStarted = false;
/**
* Stop reacting to shortcuts (unbind event handlers).
*/
$.Shortcuts.stop = function(list) {
list = list || 'default';
active[list] = false;

var has_active = false;

$.each(active, function(list, list_activated) {
if (list_activated === true) {
has_active = true
}
});

if(has_active === false){
$(document).unbind('keypress.shortcuts keydown.shortcuts keyup.shortcuts');
isStarted = false;
}
return this;
};

/**
* Return the list of active lists
* @return {Array}
*/
$.Shortcuts.getActiveLists = function() {
var activies = [];

$.each(active, function(list, list_activated) {
if (list_activated === true) {
activies.push(list);
}
});

return activies;
};

/**
* Add a shortcut.
* @param {Object} params Shortcut parameters.
Expand Down Expand Up @@ -200,7 +246,10 @@
var listNames = params.list ? params.list.replace(/\s+/g, '').split(',') : ['default'];

$.each(listNames, function(i, name) {
if (!lists[name]) { lists[name] = {}; }
if (!lists[name]) {
lists[name] = {};
active[name] = false;
}
var list = lists[name];
var masks = params.mask.toLowerCase().replace(/\s+/g, '').split(',');

Expand Down