Refactored to allow for chaining

baphled · Jun 15, 2010 · 7b34aaf · 7b34aaf
1 parent b49e932
commit 7b34aaf
Showing 1 changed file with 18 additions and 18 deletions.
diff --git a/public/javascripts/jquery-icon-animation.js b/public/javascripts/jquery-icon-animation.js
@@ -8,24 +8,6 @@
   **/
 $.fn.animateIconPanel = function(options) {
   var opts = $.extend({}, $.fn.animateIconPanel.defaults, options);
-
-  $(this).each(function() {
-    var $iconList = $('ul.icons', this),
-          $innerWrapper = $('<span>')
-            .addClass('ui-icon ui-icon-info')
-            .css({'float': 'left', 'padding-right': '0.3em'}),
-          $innerContent = $('<strong>').append('Double click'),
-          $content = $('<span> to view panel</span>'),
-          $textWrapper = $('<span>')
-            .append($innerWrapper)
-            .append($innerContent)
-            .append($content)
-            .addClass(opts.panelTextClass)
-            .css({'display': 'block', 'float': 'right'});
-
-    $iconList.hide().parent().removeClass(opts.panelClass);
-    $iconList.before($textWrapper);
-  })
 
   $(this).unbind('dbclick');
 
@@ -48,6 +30,24 @@ $.fn.animateIconPanel = function(options) {
     }
   });
 
+  return $(this).each(function() {
+    var $iconList = $('ul.icons', this),
+          $innerWrapper = $('<span>')
+            .addClass('ui-icon ui-icon-info')
+            .css({'float': 'left', 'padding-right': '0.3em'}),
+          $innerContent = $('<strong>').append('Double click'),
+          $content = $('<span> to view panel</span>'),
+          $textWrapper = $('<span>')
+            .append($innerWrapper)
+            .append($innerContent)
+            .append($content)
+            .addClass(opts.panelTextClass)
+            .css({'display': 'block', 'float': 'right'});
+
+    $iconList.hide().parent().removeClass(opts.panelClass);
+    $iconList.before($textWrapper);
+  });
+
   // private function for debugging
   function debug($obj) {
     if (window.console && window.console.log) {