# HG changeset patch
# User Takumi IINO <trot.thunder@gmail.com>
# Date 2014-11-29 08:25:13
# Node ID 26c3e4708f5877622da374d44b2f1626ffa1b6a9
# Parent  4ead043c334945cbc62fdacf19e72a115b2ac8db

javascripts: pass jQuery object to q_filter

diff --git a/kallithea/public/js/base.js b/kallithea/public/js/base.js
--- a/kallithea/public/js/base.js
+++ b/kallithea/public/js/base.js
@@ -559,19 +559,19 @@ var _close_tooltip = function(e){
  * @param display_element function that takes current node from nodes and
  *    does hide or show based on the node
  */
-var q_filter = function(target, nodes, display_element){
-    var nodes = nodes;
+var q_filter = function(target, $nodes, display_element) {
+    var $nodes = $nodes;
     var $q_filter_field = $('#' + target);
     var F = YAHOO.namespace(target);
 
-    $q_filter_field.keyup(function(e){
+    $q_filter_field.keyup(function (e) {
         clearTimeout(F.filterTimeout);
         F.filterTimeout = setTimeout(F.updateFilter, 600);
     });
 
     F.filterTimeout = null;
 
-    F.updateFilter  = function() {
+    F.updateFilter = function () {
         // Reset timeout
         F.filterTimeout = null;
 
@@ -579,23 +579,21 @@ var q_filter = function(target, nodes, d
 
         var req = $q_filter_field.val().toLowerCase();
 
-        var l = nodes.length;
-        var i;
         var showing = 0;
-
-        for (i=0; i<l; i++ ){
-            var n = nodes[i];
-            var target_element = display_element(n)
-            if(req && n.innerHTML.toLowerCase().indexOf(req) == -1){
+        $nodes.each(function () {
+            var n = this;
+            var target_element = display_element(n);
+            if (req && n.innerHTML.toLowerCase().indexOf(req) == -1) {
                 $(target_element).hide();
             }
             else{
                 $(target_element).show();
                 showing += 1;
             }
-        }
+        });
 
-        $('#repo_count').html(showing); /* FIXME: don't hardcode */
+        $('#repo_count').html(showing);
+        /* FIXME: don't hardcode */
     }
 };
 
diff --git a/kallithea/templates/bookmarks/bookmarks.html b/kallithea/templates/bookmarks/bookmarks.html
--- a/kallithea/templates/bookmarks/bookmarks.html
+++ b/kallithea/templates/bookmarks/bookmarks.html
@@ -91,7 +91,7 @@ myDataTable.subscribe('postRenderEvent',
     var func = function(node){
         return node.parentNode.parentNode.parentNode.parentNode.parentNode;
     }
-    q_filter('q_filter_bookmarks',YUQ('div.table tr td .logbooks .booktag a'),func);
+    q_filter('q_filter_bookmarks',$('div.table tr td .logbooks .booktag a'),func);
 });
 
 </script>
diff --git a/kallithea/templates/branches/branches.html b/kallithea/templates/branches/branches.html
--- a/kallithea/templates/branches/branches.html
+++ b/kallithea/templates/branches/branches.html
@@ -91,7 +91,7 @@ myDataTable.subscribe('postRenderEvent',
     var func = function(node){
         return node.parentNode.parentNode.parentNode.parentNode.parentNode;
     }
-    q_filter('q_filter_branches',YUQ('div.table tr td .logtags .branchtag a'),func);
+    q_filter('q_filter_branches',$('div.table tr td .logtags .branchtag a'),func);
 });
 
 </script>
diff --git a/kallithea/templates/tags/tags.html b/kallithea/templates/tags/tags.html
--- a/kallithea/templates/tags/tags.html
+++ b/kallithea/templates/tags/tags.html
@@ -91,7 +91,7 @@ myDataTable.subscribe('postRenderEvent',
     var func = function(node){
         return node.parentNode.parentNode.parentNode.parentNode.parentNode;
     }
-    q_filter('q_filter_tags',YUQ('div.table tr td .logtags .tagtag a'),func);
+    q_filter('q_filter_tags',$('div.table tr td .logtags .tagtag a'),func);
 });
 
 </script>