dialog layout tweaks

functions.js

@@ -1506,18 +1506,18 @@ function filterDlgCheckAction(sender) {
 			return;
 		}
 
-		var action_param = form.action_param;
+		var action_param = document.getElementById("filter_dlg_param_box");
 
 		if (!action_param) {
-			debug("filterDlgCheckAction: can't find action param!");
+			debug("filterDlgCheckAction: can't find action param box!");
 			return;
 		}
 
 		// if selected action supports parameters, enable params field
 		if (action == 4 || action == 6) {
-			action_param.disabled = false;
+			Element.show(action_param);
 		} else {
-			action_param.disabled = true;
+			Element.hide(action_param);
 		}
 
 	} catch (e) {

modules/popup-dialog.php

@@ -383,10 +383,13 @@
 	
 			print "</select>";
 
-			print " " . __("with params") . " ";
+			print "<span id=\"filter_dlg_param_box\" style=\"display : none\">";
+			print " " . __("with parameters:") . " ";
+			print "<input size=\"20\"
+					name=\"action_param\">";
+			print "</span>";
 
-			print "<input disabled size=\"20\"
-				name=\"action_param\">";
+			print "&nbsp;"; // tiny layout hack
 
 			print "</div>";
 

modules/pref-feeds.php

@@ -361,19 +361,16 @@
 			print "</form>";
 
 			print "<div class='dlgButtons'>
+				<div style=\"float : left\">
+					<input type='submit' class='button'			
+					onclick='return unsubscribeFeed($feed_id, \"$title\")' value=\"".__('Unsubscribe')."\">
+				</div>
 				<input type=\"submit\" class=\"button\" 
 				onclick=\"return feedEditSave()\" value=\"".__('Save')."\">
 				<input type='submit' class='button'			
 				onclick=\"return feedEditCancel()\" value=\"".__('Cancel')."\">
-				<div style=\"float : left\">
-
-				<input type='submit' class='button'			
-					onclick='return unsubscribeFeed($feed_id, \"$title\")' value=\"".__('Unsubscribe')."\">
-							
 				</div>";
 
-			print "</div>";
-
 			return;
 		}
 

modules/pref-filters.php

@@ -74,12 +74,15 @@
 	
 			print "</select>";
 
-			print " " . __("with params") . " ";
+			$param_hidden = ($action_id == 4 || $action_id == 6) ? "" : "display : none";
 
-			$param_disabled = ($action_id == 4 || $action_id == 6) ? "" : "disabled";
+			print "<span id=\"filter_dlg_param_box\" style=\"$param_hidden\">";
+			print " " . __("with parameters:") . " ";
+			print "<input size=\"20\"
+					name=\"action_param\" value=\"$action_param\">";
+			print "</span>";
 
-			print "<input $param_disabled size=\"20\"
-				name=\"action_param\" value=\"$action_param\">";
+			print "&nbsp;"; // tiny layout hack
 
 			print "</div>";
 
@@ -111,6 +114,14 @@
 
 			print "<div class=\"dlgButtons\">";
 
+			$reg_exp = htmlspecialchars($reg_exp); // second escaping seems to be needed for javascript
+
+			print "<div style=\"float : left\">";
+			print "<input type=\"submit\" 
+				class=\"button\" onclick='return removeFilter($filter_id, \"$reg_exp\")' 
+				value=\"".__('Remove')."\"> ";
+			print "</div>";
+
 			print "<input type=\"submit\" 
 				id=\"infobox_submit\"
 				class=\"button\" onclick=\"return filterEditSave()\" 
@@ -120,14 +131,6 @@
 				type=\"submit\" onclick=\"return filterEditCancel()\" 
 				value=\"".__('Cancel')."\">";
 
-			$reg_exp = htmlspecialchars($reg_exp); // second escaping seems to be needed for javascript
-
-			print "<div style=\"float : left\">";
-			print "<input type=\"submit\" 
-				class=\"button\" onclick='return removeFilter($filter_id, \"$reg_exp\")' 
-				value=\"".__('Remove')."\"> ";
-			print "</div>";
-
 			print "</div>";
 
 			return;