easier user privilege editor, misc js cleanups

4 files changed, 43 insertions, 19 deletions

diff --git a/backend.php b/backend.php
index 9dae91381..ba73e1a0f 100644
--- a/backend.php
+++ b/backend.php
@@ -3984,6 +3984,8 @@
 			$line["last_login"] = date(get_pref($link, 'SHORT_DATE_FORMAT'),
 				strtotime($line["last_login"]));
 
+			$access_level_names = array(0 => "User", 10 => "Administrator");
+
 /*			if ($uid == $_SESSION["uid"]) {
 
 				print "<td align='center'><input disabled=\"true\" type=\"checkbox\" 
@@ -4005,7 +4007,7 @@
 					$line["email"] . "</td>";			
 
 				print "<td><a href=\"javascript:editUser($uid);\">" . 
-					$line["access_level"] . "</td>";			
+					$access_level_names[$line["access_level"]] . "</td>";			
 
 			} else if ($uid != $edit_uid) {
 
@@ -4014,7 +4016,7 @@
 
 				print "<td>".$line["login"]."</td>";		
 				print "<td>".$line["email"]."</td>";		
-				print "<td>".$line["access_level"]."</td>";		
+				print "<td>".$access_level_names[$line["access_level"]]."</td>";
 
 			} else {
 
@@ -4027,9 +4029,23 @@
 				print "<td><input id=\"iedit_email\" value=\"".$line["email"].
 					"\"></td>";
 
-				print "<td><input id=\"iedit_ulevel\" value=\"".$line["access_level"].
-					"\"></td>";
-						
+//				print "<td><input id=\"iedit_ulevel\" value=\"".$line["access_level"].
+//					"\"></td>";
+
+				print "<td>";
+				print "<select id=\"iedit_ulevel\">";
+				foreach (array_keys($access_level_names) as $al) {
+					if ($al == $line["access_level"]) {
+						$selected = "selected";
+					} else {
+						$selected = "";
+					}					
+					print "<option $selected id=\"$al\">" . 
+						$access_level_names[$al] . "</option>";
+				}
+				print "</select>";
+				print "</td>";
+
 			}
 				
 			print "<td>".$line["last_login"]."</td>";		
diff --git a/functions.js b/functions.js
index 92bcc3bf9..1d45b1cbf 100644
--- a/functions.js
+++ b/functions.js
@@ -870,18 +870,22 @@ function leading_zero(p) {
 }
 
 function center_element(e) {
-	var c_width = document.body.clientWidth;
-	var c_height = document.body.clientHeight;
 
-	var c_scroll = document.body.scrollTop;
-
-	var e_width = e.clientWidth;
-	var e_height = e.clientHeight;
-
-	var set_y = (c_height / 2) + c_scroll - (e_height / 2);
-	var set_x = (c_width / 2) - (e_width / 2);
-
-	e.style.top = set_y + "px";
-	e.style.left = set_x + "px";
+	try {
+		var c_width = document.body.clientWidth;
+		var c_height = document.body.clientHeight;
+	
+		var c_scroll = document.body.scrollTop;
 	
+		var e_width = e.clientWidth;
+		var e_height = e.clientHeight;
+	
+		var set_y = (c_height / 2) + c_scroll - (e_height / 2);
+		var set_x = (c_width / 2) - (e_width / 2);
+	
+		e.style.top = set_y + "px";
+		e.style.left = set_x + "px";
+	} catch (e) {
+		exception_error("center_element", e);
+	}
 }
diff --git a/prefs.js b/prefs.js
index 1f74ae83d..8d15d5c22 100644
--- a/prefs.js
+++ b/prefs.js
@@ -153,7 +153,6 @@ function infobox_callback() {
 			box.innerHTML=xmlhttp.responseText;			
 			if (shadow) {
 				shadow.style.display = "block";
-				center_element(shadow);
 			} else {
 				box.style.display = "block";				
 			}
@@ -899,7 +898,10 @@ function userEditSave() {
 	}
 
 	var login = document.getElementById("iedit_ulogin").value;
-	var level = document.getElementById("iedit_ulevel").value;
+	var level = document.getElementById("iedit_ulevel");
+
+	level = level[level.selectedIndex].id;
+	
 	var email = document.getElementById("iedit_email").value;
 
 	if (login.length == 0) {
diff --git a/tt-rss.css b/tt-rss.css
index 6c85c713c..ed320a0c2 100644
--- a/tt-rss.css
+++ b/tt-rss.css
@@ -555,6 +555,8 @@ div.helpResponse {
 	background-image : url("images/shadow.png");
 	position : absolute;
 	width : 30%;
+	left : 35%;
+	top : 30%;
 	z-index : 3;
 	display : none;
 }