Merge pull request #90 from plone/maurits-fix-html-validation

Fix html validation

CHANGES.rst

@@ -21,6 +21,12 @@ Fixes:
 
 - Fix code analysis errors reported by http://jenkins.plone.org/view/Pkgs/job/package-plonetheme.barceloneta
   [gforcada]
+- Drop toolbar for anonymous users.
+- Fixed html validation:
+  - the banner role is unnecessary for element header,
+  - element main does not need a role attribute,
+  - the contentinfo role is unnecessary for element footer.
+  [maurits]
 
 
 1.6.17 (2016-02-27)

plonetheme/barceloneta/theme/index.html

@@ -21,7 +21,7 @@
     <section id="portal-toolbar">
     </section>
     <div class="outer-wrapper">
-      <header id="content-header" role="banner">
+      <header id="content-header">
         <div class="container">
           <header id="portal-top">
           </header>
@@ -47,7 +47,7 @@
         <div class="row">
           <aside id="global_statusmessage"></aside>
         </div>
-        <main id="main-container" class="row row-offcanvas row-offcanvas-right" role="main">
+        <main id="main-container" class="row row-offcanvas row-offcanvas-right">
           <div id="column1-container">
           </div>
           <div id="content-container">
@@ -57,7 +57,7 @@
         </main><!--/row-->
       </div><!--/container-->
     </div> <!--/outer-wrapper -->
-    <footer id="portal-footer-wrapper" role="contentinfo">
+    <footer id="portal-footer-wrapper">
       <div class="container" id="portal-footer"></div>
     </footer>
   </body>

plonetheme/barceloneta/theme/rules.xml

@@ -101,6 +101,7 @@
 
   <!-- toolbar -->
   <replace css:theme="#portal-toolbar" css:content-children="#edit-bar" css:if-not-content=".ajax_load" css:if-content=".userrole-authenticated" />
+  <drop css:theme="#portal-toolbar" css:if-content=".userrole-anonymous" />
   <replace css:theme="#anonymous-actions" css:content-children="#portal-personaltools-wrapper" css:if-not-content=".ajax_load" css:if-content=".userrole-anonymous" />
 
 </rules>