From: cajus <cajus@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Fri, 24 Jun 2005 09:03:29 +0000 (+0000)
Subject: Unified layout for macro/phone dialog
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=3f20f5b98138b053ec1a4de463ff10de47c704ba;p=gosa.git

Unified layout for macro/phone dialog


git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@831 594d385d-05f5-0310-b6e9-bd551577e9d8
---

diff --git a/html/images/select_macro.png b/html/images/select_macro.png
new file mode 100644
index 000000000..10d2a2d63
Binary files /dev/null and b/html/images/select_macro.png differ
diff --git a/plugins/gofon/phoneaccount/class_phoneAccount.inc b/plugins/gofon/phoneaccount/class_phoneAccount.inc
index e9ec34749..0574074ad 100644
--- a/plugins/gofon/phoneaccount/class_phoneAccount.inc
+++ b/plugins/gofon/phoneaccount/class_phoneAccount.inc
@@ -97,7 +97,7 @@ class phoneAccount extends plugin
     $ldap->search("(objectClass=goFonMacro)", array("*"));
 
     /* Add none for no macro*/
-    $this->macros['none']=_("none");    
+    $this->macros['none']=_("no macro");    
     $this->macro ="none";
 
 
@@ -140,6 +140,7 @@ class phoneAccount extends plugin
         }//is_array
       }//visible = 1
     }//while
+
     /* Go through already saved values, for a parameter */
     $tmp = split("!",$this->goFonMacro);
 
@@ -233,36 +234,38 @@ class phoneAccount extends plugin
         $str        = $default;
 
         /* in case of a combo box display a combobox with selected attr */
-        if($type == "combo"){
-          $str="<select name='".$var."' ".chkacl($this->acl, "goFonMacro").">";
-          foreach(split(":",$default) as $choice){
-            if($choosen==$choice){
-              $str.="\n<option name='".$var."' value='".$choice."' selected>".$choice."</option>";
-            }else{
-              $str.="\n<option name='".$var."' value='".$choice."'>".$choice."</option>";
+        $macrotab.= "<tr>";
+        switch ($type){
+
+          case "combo":
+            $str= "<select name='".$var."' ".chkacl($this->acl, "goFonMacro").">";
+            foreach(split(":",$default) as $choice){
+              if($choosen==$choice){
+                $str.= "\n<option name='".$var."' value='".$choice."' selected>".$choice."</option>";
+              }else{
+                $str.= "\n<option name='".$var."' value='".$choice."'>".$choice."</option>";
+              }
             }
-          }
-          $str.="</select>";
-        }
+            $str.="</select>";
+            $macrotab.= "<td>$name</td><td>$str";
+            break;
 
+          case "bool":
+            if(!$choosen){
+              $str="\n<input type='checkbox' name='".$var."' value='1'>";
+            }else{
+              $str="\n<input type='checkbox' name='".$var."' value='1' checked>";
+            }
+            $macrotab.= "<td colspan='2'>$str&nbsp;$name";
+            break;
 
-      
-        /* Display switch for true false*/
-        if($type == "bool"){
-          if(!$choosen){
-            $str="\n<input type='checkbox' name='".$var."' value='1'>";
-          }else{
-            $str="\n<input type='checkbox' name='".$var."' value='1' checked>";
-          }
-        }
+          case "string":
+            $str="<input name='".$var."' value='".$choosen."' ".chkacl($this->acl, "goFonMacro").">";
+            $macrotab.= "<td>$name</td><td>$str";
+            break;
 
-        /* display simple textfield */ 
-        if($type=="string"){
-          $str="<input name='".$var."' value='".$choosen."' ".chkacl($this->acl, "goFonMacro").">";
         }
-
-        /* create table entry*/
-        $macrotab.= "\n<tr><td>".$name."</td><td>".$str."</td></tr>"; 
+        $macrotab.= "</td</tr>";
 
       }
       $macrotab.="</table>";
diff --git a/plugins/gofon/phoneaccount/generic.tpl b/plugins/gofon/phoneaccount/generic.tpl
index b6f74be04..a094c78fc 100644
--- a/plugins/gofon/phoneaccount/generic.tpl
+++ b/plugins/gofon/phoneaccount/generic.tpl
@@ -52,8 +52,7 @@
 <br>
 <table>
 	<tr>
-		<td colspan=2>{t}Select macro{/t}
-			<img src="images/select_phone.png">
+		<td colspan=2><img src="images/select_macro.png" align="center">&nbsp;{t}Phone macro{/t}
 			 <select name="macro" onchange="document.mainform.submit()" {$telephoneNumberACL}>
 			   {html_options options=$macros selected=$macro}
 			 </select>