From f78fc416fa0c43b50017ea33a53820d39a77cc8d Mon Sep 17 00:00:00 2001 From: jlukic Date: Wed, 2 Oct 2013 15:40:47 -0400 Subject: [PATCH] moves attach events in sidebar to top level Former-commit-id: 18499d356e1d6cd7ba32914ffaab35c8365552af Former-commit-id: 8bc760ae1672bebbb75b384cab20720d5599ca8d --- src/modules/sidebar.js | 37 +++++++++++++++++-------------------- 1 file changed, 17 insertions(+), 20 deletions(-) diff --git a/src/modules/sidebar.js b/src/modules/sidebar.js index 63083aaa0..3236d104c 100755 --- a/src/modules/sidebar.js +++ b/src/modules/sidebar.js @@ -79,30 +79,27 @@ $.fn.sidebar = function(parameters) { $style = $('style[title=' + namespace + ']'); }, - attach: { - - events: function(selector, event) { - var - $toggle = $(selector) - ; - event = $.isFunction(module[event]) - ? module[event] - : module.toggle + attachEvents: function(selector, event) { + var + $toggle = $(selector) + ; + event = $.isFunction(module[event]) + ? module[event] + : module.toggle + ; + if($toggle.size() > 0) { + module.debug('Attaching sidebar events to element', selector, event); + $toggle + .off(eventNamespace) + .on('click' + eventNamespace, event) ; - if($toggle.size() > 0) { - module.debug('Attaching sidebar events to element', selector, event); - $toggle - .off(eventNamespace) - .on('click' + eventNamespace, event) - ; - } - else { - module.error(error.notFound); - } } - + else { + module.error(error.notFound); + } }, + show: function() { module.debug('Showing sidebar'); if(module.is.closed()) {