From 95f433cd099108de087e0965db2fbbafc740bc7a Mon Sep 17 00:00:00 2001
From: janw <janw@594d385d-05f5-0310-b6e9-bd551577e9d8>
Date: Tue, 1 Aug 2006 08:12:36 +0000
Subject: [PATCH] ACL toggles now working (tested) with firefox and konqueror.
 IE uses the old behavior.

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@4358 594d385d-05f5-0310-b6e9-bd551577e9d8
---
 html/include/focus.js |  2 +-
 include/class_acl.inc | 12 ++++--------
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/html/include/focus.js b/html/include/focus.js
index c5bf37490..a41a6ff05 100644
--- a/html/include/focus.js
+++ b/html/include/focus.js
@@ -205,8 +205,8 @@ function divtoggle(element) {
 
 	  if (document.poppedLayer.style.visibility == "visible") {
 		  hide (element);
+			cell.style.height="0px";
 			document.poppedLayer.style.height="0px";
-			cell.style.height="1px";
 		} else {
 		  show (element);
 			document.poppedLayer.style.height="";
diff --git a/include/class_acl.inc b/include/class_acl.inc
index f3fed5371..c2ad1d09e 100644
--- a/include/class_acl.inc
+++ b/include/class_acl.inc
@@ -487,12 +487,8 @@ class acl extends plugin
       $currentAcl= $this->aclContents[$key];
 
       /* Object header */
-      if($_SESSION['js']) {
-        if(isset($_SERVER['HTTP_USER_AGENT']) && preg_match("/firefox/i",$_SERVER['HTTP_USER_AGENT'])) {
+      if($_SESSION['js'] && (isset($_SERVER['HTTP_USER_AGENT']) && preg_match("/gecko/i",$_SERVER['HTTP_USER_AGENT']))) {
           $display.= "<table style='width:100%;border:1px solid #A0A0A0' cellspacing=0 cellpadding=2><tr><td style='background-color:#C8C8C8;height:1.8em;' colspan=".($cols-1)."><b>"._("Object").": $name</b></td><td align='right' style='background-color:#C8C8C8;height:1.8em;'><input type='button' onclick='divtoggle(\"$name\");' value='"._("Show/Hide Advanced Settings")."' /></td></tr>";
-        } else {
-          $display.= "<table style='width:100%;border:1px solid #A0A0A0' cellspacing=0 cellpadding=2><tr><td style='background-color:#C8C8C8;height:1.8em;' colspan=$cols><b>"._("Object").": $name</b></td></tr>";
-        }
       } else {
           $display.= "<table style='width:100%;border:1px solid #A0A0A0' cellspacing=0 cellpadding=2><tr><td style='background-color:#C8C8C8;height:1.8em;' colspan=$cols><b>"._("Object").": $name</b></td></tr>";
       }
@@ -525,8 +521,8 @@ class acl extends plugin
       $splist= $plist[preg_replace('%^.*/%', '', $key)]['plProvidedAcls'];
       asort($splist);
       if($_SESSION['js']) {
-        if(isset($_SERVER['HTTP_USER_AGENT']) && preg_match("/firefox/i",$_SERVER['HTTP_USER_AGENT'])) {
-          $display.= "<tr id='tr_$name' style='vertical-align:top;height:0px;'><td colspan=".$cols."><div id='$name' style='visibility:hidden;height:0px;vertical-align:top;width:100%;'><table style='width:100%;'>";
+        if(isset($_SERVER['HTTP_USER_AGENT']) && preg_match("/gecko/i",$_SERVER['HTTP_USER_AGENT'])) {
+          $display.= "<tr id='tr_$name' style='vertical-align:top;height:1px;'><td colspan=".$cols."><div id='$name' style='overflow:hidden;visibility:hidden;height:0px;vertical-align:top;width:100%;'><table style='width:100%;'>";
         }
       }
       foreach($splist as $attr => $dsc){
@@ -566,7 +562,7 @@ class acl extends plugin
       }
 
       if($_SESSION['js']) {
-        if(isset($_SERVER['HTTP_USER_AGENT']) && preg_match("/firefox/i",$_SERVER['HTTP_USER_AGENT'])) {
+        if(isset($_SERVER['HTTP_USER_AGENT']) && preg_match("/gecko/i",$_SERVER['HTTP_USER_AGENT'])) {
           $display.= "</table></div></td></tr>";
         }
       }
-- 
2.30.2