[embjopr-commits] EMBJOPR SVN: r1037 - branches/EmbJopr_1_3_4_SP1_JBPAPP-7551/core/src/main/webapp/WEB-INF/facelets.

[embjopr-commits at lists.jboss.org]

Author: loleary
Date: 2011-11-23 17:09:53 -0500 (Wed, 23 Nov 2011)
New Revision: 1037

Modified:
   branches/EmbJopr_1_3_4_SP1_JBPAPP-7551/core/src/main/webapp/WEB-INF/facelets/layout.xhtml
Log:
EMBJOPR-358: In IE7, configuration tab under bindings page is blank, works fine in FF
Issue affected IE8 in IE7 Compatibility View mode and IE7. Removed the use of the deprecated expression() function in the CSS definition for IE7.

Modified: branches/EmbJopr_1_3_4_SP1_JBPAPP-7551/core/src/main/webapp/WEB-INF/facelets/layout.xhtml
===================================================================
--- branches/EmbJopr_1_3_4_SP1_JBPAPP-7551/core/src/main/webapp/WEB-INF/facelets/layout.xhtml	2011-11-23 21:57:13 UTC (rev 1036)
+++ branches/EmbJopr_1_3_4_SP1_JBPAPP-7551/core/src/main/webapp/WEB-INF/facelets/layout.xhtml	2011-11-23 22:09:53 UTC (rev 1037)
@@ -2,7 +2,7 @@
 
 <!--
   ~ Jopr Management Platform
-  ~ Copyright (C) 2005-2010 Red Hat, Inc.
+  ~ Copyright (C) 2005-2011 Red Hat, Inc.
   ~ All rights reserved.
   ~
   ~ This program is free software; you can redistribute it and/or modify
@@ -126,26 +126,27 @@
     <ui:include src="resourceNavigation.xhtml"/>
     <ui:include src="header.xhtml"/>
     <ui:include src="footer.xhtml"/>
-
-        <!-- I'm giving up on proper CSS for IE. These poor hacks make the tab menu look nice in both IE6 and IE7 -->
-        <!-- This has to be present only *AFTER* all the content is defined, because the below CSS contains expressions
-             refering to the DOM objects defined above (like content). -->
-        <!--[if lt IE 8]>
-        <script type="text/javascript" language="javascript">
-            var scrollBarWidth = getScrollBarWidth();
-        </script>
-        <style type="text/css" media="screen">
-        h\tml #content
-            {
-            /* This definition is repeated from above so that it also applies for IE7 here, where we need this pixel-sized
-               padding so that we can compute the width of the tabmenu below. */
-            padding: 16px; /* replace the em units for padding with something we can compute with in the expressions below */
-            }
-        #tabmenu {
-            width: expression((document.body.clientWidth - 300 - 2 * 16 - (content.scrollHeight > content.clientHeight ? scrollBarWidth : 0)) + 'px'); /* 300 = width of sidebar, 16 content padding. */
-            }
-        </style>
-        <![endif]-->
+    <!-- I'm giving up on proper CSS for IE. These poor hacks make the tab menu look nice in both IE6 and IE7 -->
+    <!-- This has to be present only *AFTER* all the content is defined, because the below CSS contains expressions
+        refering to the DOM objects defined above (like content). -->
+    <!--[if lt IE 8]>
+    <script type="text/javascript" language="javascript">
+       window.onload = resizeTabMenu;
+       window.onresize = resizeTabMenu;
+       function resizeTabMenu(){
+          var scrollBarWidth = (content.scrollHeight > content.clientHeight ? getScrollBarWidth() : 0);
+          tabmenu.style.width = (document.body.clientWidth - 300 - (2 * 16) - scrollBarWidth) + 'px';
+       }
+    </script>
+    <style type="text/css" media="screen">
+    h\tml #content
+       {
+       /* This definition is repeated from above so that it also applies for IE7 here, where we need this pixel-sized
+          padding so that we can compute the width of the tabmenu below. */
+       padding: 16px; /* replace the em units for padding with something we can compute with in the expressions below */
+       }
+    </style>
+    <![endif]-->
     </body>
 </f:view>
 </html>