You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dw...@apache.org on 2008/05/20 19:53:36 UTC

svn commit: r658362 - in /geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp: WEB-INF/themes/banner.jsp WEB-INF/themes/default-theme.jsp main.css

Author: dwoods
Date: Tue May 20 10:53:36 2008
New Revision: 658362

URL: http://svn.apache.org/viewvc?rev=658362&view=rev
Log:
GERONIMO-4024 Accessbility issue: no 'Skip to main content' support for screen readers.  Also, fixed indent usage in banner.jsp.

Modified:
    geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/WEB-INF/themes/banner.jsp
    geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/WEB-INF/themes/default-theme.jsp
    geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/main.css

Modified: geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/WEB-INF/themes/banner.jsp
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/WEB-INF/themes/banner.jsp?rev=658362&r1=658361&r2=658362&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/WEB-INF/themes/banner.jsp (original)
+++ geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/WEB-INF/themes/banner.jsp Tue May 20 10:53:36 2008
@@ -1,5 +1,5 @@
 <%@ page language="java"%>
- <%--
+<%--
 Licensed to the Apache Software Foundation (ASF) under one or more
 contributor license agreements.  See the NOTICE file distributed with
 this work for additional information regarding copyright ownership.
@@ -19,16 +19,16 @@
 --%>
   <tr>
     <td>
-	  <table width="100%" height="86"  border="0" cellpadding="0" cellspacing="0">
+      <table width="100%" height="86"  border="0" cellpadding="0" cellspacing="0">
         <tr>
           <td height="86" class="Logo" border="0"></td>
-          <td height="86" class="Top" border="0">&nbsp; </TD>
+          <td height="86" class="Top" border="0">&nbsp;</td>
           <td height="86" class="Top" border="0" width="40"></td>
           <td height="86" class="Top" border="0" width="40">
-<a href="<%=request.getContextPath()%>/logout.jsp"><img border="0" src="<%=request.getContextPath()%>/images/head_logout_63x86.gif" alt="Logout"></a>
+<a href="<%=request.getContextPath()%>/logout.jsp"><img border="0" src="<%=request.getContextPath()%>/images/head_logout_63x86.gif" alt="Logout"/></a>
           </td>
         </tr>
       </table>
-	</td>
+    </td>
   </tr>
 

Modified: geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/WEB-INF/themes/default-theme.jsp
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/WEB-INF/themes/default-theme.jsp?rev=658362&r1=658361&r2=658362&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/WEB-INF/themes/default-theme.jsp (original)
+++ geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/WEB-INF/themes/default-theme.jsp Tue May 20 10:53:36 2008
@@ -25,7 +25,20 @@
 
 <html>
 <%@ include file="./head.jsp" %>
-<body marginwidth="0" marginheight="0" leftmargin="0" topmargin="0" rightmargin="0">
+<body id="admin-console" marginwidth="0" marginheight="0" leftmargin="0" topmargin="0" rightmargin="0">
+
+<!-- start accessibility prolog -->
+<div class="skip"><a href="#left-nav" accesskey="1">Skip to navigation</a></div>
+<div class="skip"><a href="#content" accesskey="2">Skip to main content</a></div>
+<div id="access-info">
+    <p class="access" >The access keys for this page are:</p>
+    <ul class="access">
+        <li>ALT plus 1 skips to navigation.</li>
+        <li>ALT plus 2 skips to main content.</li>
+    </ul>
+</div>
+<!-- end accessibility prolog -->
+
 <table width="100%" cellpadding="0" cellspacing="0" border="0" id="rootfragment">
 
     <!-- Header -->
@@ -48,7 +61,7 @@
                     <!-- Navigation Column -->
                     <td class="Gutter">&nbsp;</td> <!-- Spacer -->
                     <td width="200px" valign="top"> 
-                        <div class="Menu"> 
+                        <div class="Menu" id="left-nav"> 
                             <table width="100%"  border="0" cellpadding="0" cellspacing="0"> 
                                 <tr> 
                                     <td>
@@ -64,7 +77,7 @@
                     <td class="Gutter">&nbsp;</td> <!-- Spacer -->
                     <td valign="top">
                         <!-- Content block: portlets are divided into two columns/groups -->
-                         <!--<div id="body-block" style="height:100%">-->
+                          <!--<div id="body-block" style="height:100%">-->
                             <div id="content">
                                 <pluto:isMaximized var="isMax"/>
                                 <c:forEach var="portlet" varStatus="status" items="${currentPage.portletIds}">
@@ -84,4 +97,4 @@
     </tr>
 </table>
 </body>
-</html>
\ No newline at end of file
+</html>

Modified: geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/main.css
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/main.css?rev=658362&r1=658361&r2=658362&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/main.css (original)
+++ geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/main.css Tue May 20 10:53:36 2008
@@ -505,3 +505,32 @@
     border: thin solid #000000;
 }
 
+.access
+{
+    display: none;
+    position: absolute;
+    left: -3000px;
+    width: 500px;
+}
+
+.skip a {
+    position: absolute;
+    overflow: hidden;
+    width: 0;
+    height: 0;
+}
+
+.skip a:active, .skip a:focus {
+    position: absolute;
+    overflow: visible;
+    width: 135px;
+    height: auto;
+    background-color: #FFFFFF;
+    color: #000000;
+    top: 75px;
+    left: 5px;
+    z-index:1000;
+    padding: 5px;
+    border: 2px solid #000000;
+    font-weight: bold;
+}