Index: trunk/admin/cat_select.php
===================================================================
diff -u -r101 -r251
--- trunk/admin/cat_select.php	(.../cat_select.php)	(revision 101)
+++ trunk/admin/cat_select.php	(.../cat_select.php)	(revision 251)
@@ -162,110 +162,101 @@
   require_once($pathtoroot.$admin."/include/mainscript.php");
 
 print <<<END
-  <script src="$browseURL/toolbar.js"></script>
-  <script src="$browseURL/checkboxes_new.js"></script>
-  <script language="JavaScript1.2" src="$browseURL/fw_menu.js"></script>
-  <script language="JavaScript1.2">
-      var enableContextMenus= true;
-      var doubleClickAction = 'select';
-function toggleTabB(tabId, atm)
-{
-    var hl = document.getElementById("hidden_line");
-   var activeTabId;
+<script src="$browseURL/toolbar.js"></script>
+<script src="$browseURL/checkboxes_new.js"></script>
+<script language="JavaScript1.2" src="$browseURL/fw_menu.js"></script>
+<script language="JavaScript1.2">
+	var enableContextMenus= true;
+    var doubleClickAction = 'select';
+	
+	function toggleTabB(tabId, atm)
+	{
+    	var hl = document.getElementById("hidden_line");
+   		var activeTabId;
 
-   if (activeTab) activeTabId = activeTab.id;
-   if (activeTabId == tabId)
-   {
-      var devider = document.getElementById("tabsDevider");
-      devider.style.display = "";
+   		if (activeTab) activeTabId = activeTab.id;
+   		if (activeTabId == tabId)
+		{
+      		var devider = document.getElementById("tabsDevider");
+      		devider.style.display = "";
 
-      unselectAll(tabId);
-      var tab = document.getElementById(tabId);
-      tab.active = false;
-      activeTab = null;
-      collapseTab = tab;
-      toolbar.setTab(null);
-      showTab();
-   }
+			unselectAll(tabId);
+			var tab = document.getElementById(tabId);
+			tab.active = false;
+			activeTab = null;
+			collapseTab = tab;
+			toolbar.setTab(null);
+			showTab();
+   		}
+   		else 
+   		{   
+      		if (activeTab)
+      			toggleTab(tabId, true) 
+      		else 
+      			toggleTab(tabId, atm)
+      		if (hl) hl.style.display = "none";
+   		}
+   		tab_hdr = document.getElementById('tab_headers');
+   		if (!tab_hdr) return; 
+		
+		// process all module tabs
+	   	var active_str = '';
+   		for(var i = 0; i < tabIDs.length; i++)
+   		{
+      		var tabHeader;
+      		TDs = tab_hdr.getElementsByTagName("TD");
+   			for (var j = 0; j < TDs.length; j++)
+      			if (TDs[j].getAttribute("tabHeaderOf") == tabIDs[i])
+      			{
+         			tabHeader = TDs[j];
+         			break;
+      			}
+      			if (!tabHeader) continue;
 
-   else 
-   {   
-      if (activeTab) toggleTab(tabId, true) 
-      else toggleTab(tabId, atm)
+      			var tab = document.getElementById(tabIDs[i]);
+      			if (!tab) continue;
+				active_str = (tab.active) ? "tab_active" : "tab_inactive";
+				
+      			if (TDs[j].getAttribute("tabHeaderOf") == tabId) {
+	      			// module tab is selected
+	      			SetBackground('l_' + tabId, "$imagesURL/itemtabs/" + active_str + "_l.gif");
+	        		SetBackground('m_' + tabId, "$imagesURL/itemtabs/" + active_str + ".gif");
+	      			SetBackground('m1_' + tabId, "$imagesURL/itemtabs/" + active_str + ".gif");
+	        		SetBackground('r_' + tabId, "$imagesURL/itemtabs/" + active_str + "_r.gif");
+	      		}
+	      		else 
+	      		{
+	      			// module tab is not selected
+	      			SetBackground('l_' +tabIDs[i], "$imagesURL/itemtabs/" + active_str + "_l.gif");
+	        		SetBackground('m_' + tabIDs[i], "$imagesURL/itemtabs/" + active_str + ".gif");
+	      			SetBackground('m1_' + tabIDs[i], "$imagesURL/itemtabs/" + active_str + ".gif");
+	        		SetBackground('r_' + tabIDs[i], "$imagesURL/itemtabs/" + active_str + "_r.gif");
+	      		}
 
-      if (hl) hl.style.display = "none";
-   }
-   tab_hdr = document.getElementById('tab_headers');
-   if (!tab_hdr) return; 
+				var images = tabHeader.getElementsByTagName("IMG");
+				if (images.length < 1) continue;
 
-   for (var i = 0; i < tabIDs.length; i++)
-   {
-      var tabHeader;
-      TDs = tab_hdr.getElementsByTagName("TD");
-   for (var j = 0; j < TDs.length; j++)
-      if (TDs[j].getAttribute("tabHeaderOf") == tabIDs[i])
-      {
-         tabHeader = TDs[j];
-         break;
-      }
-      if (!tabHeader) continue;
+				images[0].src = "$imagesURL/itemtabs/" + ((tab.active) ? "divider_up" : "divider_empty") + ".gif";
+   		}
+	}
 
-      var tab = document.getElementById(tabIDs[i]);
-      if (!tab) continue;
-
-      if (TDs[j].getAttribute("tabHeaderOf") == tabId) {
-      	cell = document.getElementById('l_' + tabId);
-      	cell.background = "$imagesURL/itemtabs/" + ((tab.active) ? "tab_active_l" : "tab_inactive_l") + ".gif"
-      
-      	cell = document.getElementById('m_' + tabId);
-      	cell.background = "$imagesURL/itemtabs/" + ((tab.active) ? "tab_active" : "tab_inactive") + ".gif"      
-      
-      	cell = document.getElementById('m1_' + tabId);
-      	cell.background = "$imagesURL/itemtabs/" + ((tab.active) ? "tab_active" : "tab_inactive") + ".gif"      
-      
-      	cell = document.getElementById('r_' + tabId);
-      	cell.background = "$imagesURL/itemtabs/" + ((tab.active) ? "tab_active_r" : "tab_inactive_r") + ".gif"      
-      }
-      else {
-      	cell = document.getElementById('l_' + tabIDs[i]);
-      	cell.background = "$imagesURL/itemtabs/" + ((tab.active) ? "tab_active_l" : "tab_inactive_l") + ".gif"
-      
-      	cell = document.getElementById('m_' + tabIDs[i]);
-      	cell.background = "$imagesURL/itemtabs/" + ((tab.active) ? "tab_active" : "tab_inactive") + ".gif"      
-      
-      	cell = document.getElementById('m1_' + tabIDs[i]);
-      	cell.background = "$imagesURL/itemtabs/" + ((tab.active) ? "tab_active" : "tab_inactive") + ".gif"      
-      
-      	cell = document.getElementById('r_' + tabIDs[i]);
-      	cell.background = "$imagesURL/itemtabs/" + ((tab.active) ? "tab_active_r" : "tab_inactive_r") + ".gif"            
-      }
-           
-
-//      tabHeader.setAttribute("background", "$imagesURL/itemtabs/" + ((tab.active) ? "tab_active" : "tab_inactive") + ".gif")
-
-      var images = tabHeader.getElementsByTagName("IMG");
-      if (images.length < 1) continue;
-
-      images[0].src = "$imagesURL/itemtabs/" + ((tab.active) ? "divider_up" : "divider_empty") + ".gif";
-   }
-}
-
-function toggleCategoriesB(tabHeader, instant)
-{
-   var categories = document.getElementById('categories');
-   if (!categories) return;
-   toggleCategories(instant);
+	function toggleCategoriesB(tabHeader, instant)
+	{
+		var categories = document.getElementById('categories');
+		if (!categories) return;
+		toggleCategories(instant);
+		
+		var active_str = '$imagesURL'+'/itemtabs/' + (categories.active ? 'tab_active' : 'tab_inactive');
+		SetBackground('l_cat', active_str + '_l.gif');
+		SetBackground('m_cat', active_str + '.gif');
+		SetBackground('m1_cat', active_str + '.gif');
+		SetBackground('r_cat', active_str + '_r.gif');
 	
-   document.all.l_cat.background = '$imagesURL'+'/itemtabs/' + ((categories.active) ? "tab_active_l" : "tab_inactive_l") + ".gif"
-   document.all.m_cat.background = '$imagesURL'+'/itemtabs/' + ((categories.active) ? "tab_active" : "tab_inactive") + ".gif"
-   document.all.m1_cat.background = '$imagesURL'+'/itemtabs/' + ((categories.active) ? "tab_active" : "tab_inactive") + ".gif"
-   document.all.r_cat.background = '$imagesURL'+'/itemtabs/' + ((categories.active) ? "tab_active_r" : "tab_inactive_r") + ".gif"
-   //tabHeader.setAttribute("background", '$imagesURL'+'/itemtabs/' + ((categories.active) ? "tab_active" : "tab_inactive") + ".gif")
-   var images = tabHeader.getElementsByTagName("IMG");
-   if (images.length < 1) return;
-     images[0].src = '$imagesURL'+'/itemtabs/' + ((categories.active) ? "divider_up" : "divider_dn") + ".gif";
-}
-  </script>  
+	   	var images = tabHeader.getElementsByTagName("IMG");
+	   	if (images.length < 1) return;
+			images[0].src = '$imagesURL'+'/itemtabs/' + ((categories.active) ? "divider_up" : "divider_dn") + ".gif";
+	}
+</script>  
   <link rel="stylesheet" type="text/css" href="$browseURL/checkboxes.css">
   <link rel="stylesheet" type="text/css" href="$cssURL/style.css">
   <link rel="stylesheet" type="text/css" href="$browseURL/toolbar.css">
@@ -337,21 +328,29 @@
       $objCatList->LoadCategories("ParentId=".$objCatList->CurrentCategoryID()." ".$CategoryFilter,$OrderBy);
 ?>
 <table cellspacing="0" cellpadding="0" width="100%" border="0">
-   <tbody>
-   <tr>
-<td width="138" height="20" nowrap="nowrap" 
-          class="active_tab" onclick="toggleCategoriesB(this)" style="" id="cats_tab">
-          <table cellspacing="0" cellpadding="0" width="100%"><tr>
-          	<td id="l_cat" background="<?php echo $imagesURL; ?>/itemtabs/tab_active_l.gif"><img height="20" src="<?php echo $imagesURL; ?>/itemtabs/divider_up.gif" width="20" border="0" align="absmiddle"></td>
-          	<td id="m_cat" nowrap background="<?php echo $imagesURL; ?>/itemtabs/tab_active.gif">
-             <?php echo admin_language("la_ItemTab_Categories"); ?>: </td>
-             <td id="m1_cat" align="right" valign="top" background="<?php echo $imagesURL; ?>/itemtabs/tab_active.gif"><span class="cats_stats">(<?php echo $objCatList->QueryItemCount; ?>)</span>&nbsp;</td>
-             <td id="r_cat" width="21" background="<?php echo $imagesURL; ?>/itemtabs/tab_active_r.gif">&nbsp;&nbsp;&nbsp;</td>
-          </tr></table>
-      </td>
-      <td>&nbsp;</td>
-   </tr>
-  </tbody>
+	<tbody>
+	<tr>
+		<td width="138" height="20" nowrap="nowrap" class="active_tab" onclick="toggleCategoriesB(this)" id="cats_tab">
+			<table cellspacing="0" cellpadding="0" width="100%" border="0">
+				<tr>
+          		<td id="l_cat" background="<?php echo $imagesURL; ?>/itemtabs/tab_active_l.gif" class="left_tab">
+          			<img src="<?php echo $imagesURL; ?>/itemtabs/divider_up.gif" width="20" height="20" border="0" align="absmiddle">
+					</td>
+          		<td id="m_cat" nowrap background="<?php echo $imagesURL; ?>/itemtabs/tab_active.gif" class="tab_class">
+             		<?php echo admin_language("la_ItemTab_Categories"); ?>:&nbsp;
+					</td>
+             	<td id="m1_cat" align="right" valign="top" background="<?php echo $imagesURL; ?>/itemtabs/tab_active.gif" class="tab_class">
+						<span class="cats_stats">(<?php echo $objCatList->QueryItemCount; ?>)</span>&nbsp;
+					</td>
+					<td id="r_cat" background="<?php echo $imagesURL; ?>/itemtabs/tab_active_r.gif" class="right_tab">
+						<img src="<?php echo $imagesURL; ?>/spacer.gif" width="21" height="20">
+					</td>
+				</tr>
+			</table>
+		</td>
+		<td>&nbsp;</td>
+	</tr>
+  	</tbody>
 </table>
 
 <div class="divider" style="" id="categoriesDevider"><img width=1 height=1 src="images/spacer.gif"></div>