From embjopr-commits at lists.jboss.org Wed Nov 23 16:49:59 2011
Content-Type: multipart/mixed; boundary="===============2108763569027085800=="
MIME-Version: 1.0
From: embjopr-commits at lists.jboss.org
To: embjopr-commits at lists.jboss.org
Subject: [embjopr-commits] EMBJOPR SVN: r1035 -
 branches/AdminConsole_EAP_5_1/core/src/main/webapp/WEB-INF/facelets.
Date: Wed, 23 Nov 2011 16:49:58 -0500
Message-ID: <201111232149.pANLnwl9021696@svn01.web.mwc.hst.phx2.redhat.com>

--===============2108763569027085800==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

Author: loleary
Date: 2011-11-23 16:49:58 -0500 (Wed, 23 Nov 2011)
New Revision: 1035

Modified:
   branches/AdminConsole_EAP_5_1/core/src/main/webapp/WEB-INF/facelets/layo=
ut.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/AdminConsole_EAP_5_1/core/src/main/webapp/WEB-INF/facele=
ts/layout.xhtml
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- branches/AdminConsole_EAP_5_1/core/src/main/webapp/WEB-INF/facelets/lay=
out.xhtml	2011-11-23 21:47:30 UTC (rev 1034)
+++ branches/AdminConsole_EAP_5_1/core/src/main/webapp/WEB-INF/facelets/lay=
out.xhtml	2011-11-23 21:49:58 UTC (rev 1035)
@@ -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=3D"resourceNavigation.xhtml"/>
     <ui:include src=3D"header.xhtml"/>
     <ui:include src=3D"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 define=
d, because the below CSS contains expressions
-             refering to the DOM objects defined above (like content). -->
-        <!--[if lt IE 8]>
-        <script type=3D"text/javascript" language=3D"javascript">
-            var scrollBarWidth =3D getScrollBarWidth();
-        </script>
-        <style type=3D"text/css" media=3D"screen">
-        h\tml #content
-            {
-            /* This definition is repeated from above so that it also appl=
ies for IE7 here, where we need this pixel-sized
-               padding so that we can compute the width of the tabmenu bel=
ow. */
-            padding: 16px; /* replace the em units for padding with someth=
ing 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 =3D 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, b=
ecause the below CSS contains expressions
+        refering to the DOM objects defined above (like content). -->
+    <!--[if lt IE 8]>
+    <script type=3D"text/javascript" language=3D"javascript">
+       window.onload =3D resizeTabMenu;
+       window.onresize =3D resizeTabMenu;
+       function resizeTabMenu(){
+          var scrollBarWidth =3D (content.scrollHeight > content.clientHei=
ght ? getScrollBarWidth() : 0);
+          tabmenu.style.width =3D (document.body.clientWidth - 300 - (2 * =
16) - scrollBarWidth) + 'px';
+       }
+    </script>
+    <style type=3D"text/css" media=3D"screen">
+    h\tml #content
+       {
+       /* This definition is repeated from above so that it also applies f=
or 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 w=
e can compute with in the expressions below */
+       }
+    </style>
+    <![endif]-->
     </body>
 </f:view>
 </html>


--===============2108763569027085800==--