From 814278b4ec0c2e24bbf4a4879b7a2018d364b124 Mon Sep 17 00:00:00 2001
From: Dobrica Pavlinusic <dpavlin@rot13.org>
Date: Wed, 30 Jun 2010 23:58:11 +0200
Subject: [PATCH 1/1] don't mess with tabindex

and don't trigger apply_regex on empty value
---
 templates/all_checkboxes.html.ep | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/templates/all_checkboxes.html.ep b/templates/all_checkboxes.html.ep
index 3b36fcf..e5c9e95 100644
--- a/templates/all_checkboxes.html.ep
+++ b/templates/all_checkboxes.html.ep
@@ -4,9 +4,9 @@
 <a id="none" href="#">none</a>
 
 <label for=filter_regex>
-<input tabindex=1 type=text name=filter_regex title="enter regex to filter values" placeholder="regex">
-<a tabindex=2 id="included" href="#" style="display:none">included</a>
-<a tabindex=3 id="excluded" href="#" style="display:none">excluded</a>
+<input  type=text name=filter_regex title="enter regex to filter values" placeholder="regex">
+<a id="included" href="#" style="display:none">included</a>
+<a id="excluded" href="#" style="display:none">excluded</a>
 </label>
 </span>
 
@@ -70,8 +70,12 @@ $('a#excluded').click( function(){
 // regex for column names
 
 var apply_regex = function(e) {
-	console.debug( this, e );
-	var r = new RegExp( this.value ,'i');
+	var user_regex = this.value;
+	console.debug( this, e, user_regex );
+
+	if ( user_regex == '' ) return false;
+
+	var r = new RegExp( user_regex,'i');
 	var included = 0;
 	var excluded = 0;
     $("input[type='checkbox']:not([disabled='disabled'])").each( function(i,element){
-- 
2.20.1