fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] changelog2012-07-12 10:25 am

[dw-free] Apply WAI-ARIA roles where appropriate to the site skin structure

[commit: http://hg.dwscoalition.org/dw-free/rev/38d66572a03f]

http://bugs.dwscoalition.org/show_bug.cgi?id=4361

WAI-ARIA role landmarks for dw-free site schemes.

Patch by [personal profile] deborah.

Files modified:
  • schemes/celerity.tt
  • schemes/common.tt
  • schemes/lynx.tt
--------------------------------------------------------------------------------
diff -r ebc1237c3bc5 -r 38d66572a03f schemes/celerity.tt
--- a/schemes/celerity.tt	Thu Jul 12 18:24:21 2012 +0800
+++ b/schemes/celerity.tt	Thu Jul 12 18:28:58 2012 +0800
@@ -40,28 +40,28 @@
     <body [% sections.bodyopts %]>
         <div id="canvas">
             <div id="page">
-                <div id="masthead">
+                <div id="masthead" role="banner">
                         [% PROCESS block.logo %]
                 </div><!-- end masthead-->
 
-                <div id="content" [% sections.contentopts %]>
+                <div id="content" role="main" [% sections.contentopts %]>
                 <h1>[% sections.title %]</h1>
                 [% content %]
                 </div><!--end content-->
                 <div id="page-decoration"></div>
                 </div><!-- end page-->
-                <div id="account-links">
+                <div id="account-links" role="navigation">
                     [% PROCESS block.accountlinks %]
                 </div><!-- end account links-->
-                <div id="sidebar">
+                <div id="sidebar" role="navigation">
                     [% PROCESS block.userpic %]
                     [% PROCESS block.menunav %]
                 </div>
                 <div id="header-divider"> <div id="header-divider-insert"></div></div>
-                <div id="header-search">
+                <div id="header-search" role="search">
                     [% dw_scheme.search_render %]
                 </div><!-- end header-search-->
-                <div id="footer">
+                <div id="footer" role="contentinfo">
                     [% PROCESS block.footer %]
                 </div>
         </div> <!-- end canvas-->
diff -r ebc1237c3bc5 -r 38d66572a03f schemes/common.tt
--- a/schemes/common.tt	Thu Jul 12 18:24:21 2012 +0800
+++ b/schemes/common.tt	Thu Jul 12 18:28:58 2012 +0800
@@ -125,26 +125,26 @@
 
 [%- BLOCK block.pagediv -%]
 <div id="page">
-    <div id="masthead">
+    <div id="masthead" role="banner">
         <span id="logo">
             [% PROCESS block.logo %]
         </span>
     </div>
 
-    <div id="content" [% sections.contentopts %]>
+    <div id="content" role="main" [% sections.contentopts %]>
     <h1>[% sections.title %]</h1>
     [% content %]
     </div>
-    <div id="account-links">
+    <div id="account-links" role="navigation">
         [% PROCESS block.accountlinks %]
     </div>
-    <div id="menu">
+    <div id="menu" role="navigation">
         [% PROCESS block.menunav %]
     </div>
-    <div id="header-search">
+    <div id="header-search" role="search">
         [% dw_scheme.search_render %]
     </div>
-    <div id="footer">
+    <div id="footer" role="contentinfo">
         [% PROCESS block.footer %]
     </div>
 </div>
diff -r ebc1237c3bc5 -r 38d66572a03f schemes/lynx.tt
--- a/schemes/lynx.tt	Thu Jul 12 18:24:21 2012 +0800
+++ b/schemes/lynx.tt	Thu Jul 12 18:28:58 2012 +0800
@@ -20,10 +20,12 @@
 
 <body [% sections.bodyopts %]>
 
+<div role="main">
 [% content %]
+</div>
 
 <hr />
-<p>[ <a href='[% site.root %]/'>[% 'lynx.nav.home' | ml %]</a> | <a href='[% site.root %]/update'>[% 'lynx.nav.update' | ml %]</a> |
+<div role="navigation><p>[ <a href='[% site.root %]/'>[% 'lynx.nav.home' | ml %]</a> | <a href='[% site.root %]/update'>[% 'lynx.nav.update' | ml %]</a> |
 [%- IF remote %][% baseurl = remote.journal_base %]
 <a href='[% baseurl %]/'>[% 'lynx.nav.recent' | ml %]</a> | <a href='[% baseurl %]/read'>[% 'lynx.nav.friends' | ml %]</a> |
 <a href='[% site.root %]/logout'>[% 'lynx.nav.logout' | ml %]</a> | [% remote.ljuser_display %] |
@@ -32,7 +34,7 @@
 [%- END %]
 <a href='[% site.root %]/tools/search'>[% 'lynx.nav.search' | ml %]</a> | 
 <a href='[% site.root %]/manage/settings/'>[% 'lynx.nav.siteopts' | ml %]</a> | 
-<a href='[% site.root %]/site/'>[% 'lynx.nav.sitemap' | ml %]</a> ]</p>
+<a href='[% site.root %]/site/'>[% 'lynx.nav.sitemap' | ml %]</a> ]</p></div>
 [% dw_scheme.final_body_html %]
 </body>
 </html>
--------------------------------------------------------------------------------