rework filter editor/add infobox layout, misc fixes

backend.php

@@ -2132,12 +2132,14 @@
 			print "<tr><td>Match:</td>
 				<td><input onkeypress=\"return filterCR(event)\"
 					 onkeyup=\"toggleSubmitNotEmpty(this, 'infobox_submit')\"
-					name=\"reg_exp\" size=\"30\" value=\"$reg_exp\">&nbsp;";
+					name=\"reg_exp\" class=\"iedit\" value=\"$reg_exp\">";
 			
-			print_select_hash("filter_type", $filter_type, $filter_types);	
+			print "</td><td>";
+			
+			print_select_hash("filter_type", $filter_type, $filter_types, "class=\"iedit\"");	
 	
 			print "</td></tr>";
-			print "<tr><td>Feed:</td><td>";
+			print "<tr><td>Feed:</td><td colspan='2'>";
 
 			print_feed_select($link, "feed_id", $feed_id);
 			
@@ -2145,7 +2147,7 @@
 	
 			print "<tr><td>Action:</td>";
 	
-			print "<td><select name=\"action_id\">";
+			print "<td colspan='2'><select name=\"action_id\">";
 	
 			$result = db_query($link, "SELECT id,description FROM ttrss_filter_actions 
 				ORDER BY name");
@@ -2254,6 +2256,7 @@
 		print "<input type=\"submit\" 
 			class=\"button\" 
 			onclick=\"javascript:displayDlg('quickAddFilter', false)\" 
+			id=\"create_filter_btn\"
 			value=\"Create filter\">"; 
 
 		$result = db_query($link, "SELECT 
@@ -2346,22 +2349,14 @@
 	
 			print "<p id=\"filterOpToolbar\">";
 	
-			if ($subop == "edit") {
-				print "Edit filter:
-					<input type=\"submit\" class=\"button\" 
-						onclick=\"javascript:filterEditSave()\" value=\"Save\">
-					<input type=\"submit\" class=\"button\" 
-						onclick=\"javascript:filterEditCancel()\" value=\"Cancel\">";
-						
-			} else {
-	
-				print "
+			print "
 					Selection:
 				<input type=\"submit\" class=\"button\" disabled=\"true\"
 					onclick=\"javascript:editSelectedFilter()\" value=\"Edit\">
 				<input type=\"submit\" class=\"button\" disabled=\"true\"
 					onclick=\"javascript:removeSelectedFilters()\" value=\"Remove\">";
-			}			
+
+			print "</p>";
 
 		} else {
 
@@ -2793,12 +2788,13 @@
 			print "<tr><td>Match:</td>
 				<td><input onkeypress=\"return filterCR(event)\"
 					 onkeyup=\"toggleSubmitNotEmpty(this, 'infobox_submit')\"
-					name=\"reg_exp\" size=\"30\">&nbsp;";
-			
-			print_select_hash("filter_type", 1, $filter_types);	
+					name=\"reg_exp\" class=\"iedit\">";		
+			print "</td><td>";
+		
+			print_select_hash("filter_type", 1, $filter_types, "class=\"iedit\"");	
 	
 			print "</td></tr>";
-			print "<tr><td>Feed:</td><td>";
+			print "<tr><td>Feed:</td><td colspan='2'>";
 
 			print_feed_select($link, "feed_id");
 			
@@ -2806,7 +2802,7 @@
 	
 			print "<tr><td>Action:</td>";
 	
-			print "<td><select name=\"action_id\">";
+			print "<td colspan='2'><select name=\"action_id\">";
 	
 			$result = db_query($link, "SELECT id,description FROM ttrss_filter_actions 
 				ORDER BY name");

functions.js

@@ -364,6 +364,11 @@ function disableContainerChildren(id, disable, doc) {
 
 	var container = doc.getElementById(id);
 
+	if (!container) {
+		//alert("disableContainerChildren: element " + id + " not found");
+		return;
+	}
+
 	for (var i = 0; i < container.childNodes.length; i++) {
 		var child = container.childNodes[i];
 

prefs.js

@@ -338,6 +338,8 @@ function editFilter(id) {
 		return
 	}
 
+	document.getElementById("create_filter_btn").disabled = true;
+
 	selectTableRowsByIdPrefix('prefFilterList', 'FILRR-', 'FICHK-', false);
 	selectTableRowById('FILRR-'+id, 'FICHK-'+id, true);
 
@@ -705,6 +707,8 @@ function filterEditCancel() {
 		printLockingError();
 		return
 	}
+
+	document.getElementById("create_filter_btn").disabled = false;
 	
 	selectPrefRows('filter', false); // cleanup feed selection
 	closeInfoBox();
@@ -760,7 +764,9 @@ function userEditSave() {
 	}
 	
 	notify("Saving user...");
-	
+
+	closeInfoBox();
+
 	var query = Form.serialize("user_edit_form");
 	
 	xmlhttp.open("GET", "backend.php?" + query, true);			
@@ -789,6 +795,8 @@ function filterEditSave() {
 
 	closeInfoBox();
 
+	document.getElementById("create_filter_btn").disabled = false;
+
 	xmlhttp.open("GET", "backend.php?" + query, true);
 	xmlhttp.onreadystatechange=filterlist_callback;
 	xmlhttp.send(null);