From 94a82049298730b6ca46924a2c71d7bcc8af63b3 Mon Sep 17 00:00:00 2001
From: markseu <mark2011@mayberg.se>
Date: Thu, 30 Aug 2018 11:06:17 +0200
Subject: [PATCH] Accessibility improvements for screen readers, thanks Steffen

---
 system/themes/snippets/content-default.php    | 2 +-
 system/themes/snippets/footer.php             | 2 +-
 system/themes/snippets/header.php             | 2 +-
 system/themes/snippets/navigation-sidebar.php | 2 +-
 system/themes/snippets/navigation-tree.php    | 2 +-
 system/themes/snippets/navigation.php         | 2 +-
 system/themes/snippets/pagination.php         | 2 +-
 system/themes/snippets/sidebar.php            | 4 ++--
 8 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/system/themes/snippets/content-default.php b/system/themes/snippets/content-default.php
index bdca389..140de88 100644
--- a/system/themes/snippets/content-default.php
+++ b/system/themes/snippets/content-default.php
@@ -1,6 +1,6 @@
 <div class="content">
 <?php $yellow->snippet("sidebar") ?>
-<div class="main">
+<div class="main" role="main">
 <h1><?php echo $yellow->page->getHtml("titleContent") ?></h1>
 <?php echo $yellow->page->getContent() ?>
 </div>
diff --git a/system/themes/snippets/footer.php b/system/themes/snippets/footer.php
index a19f026..cca8115 100644
--- a/system/themes/snippets/footer.php
+++ b/system/themes/snippets/footer.php
@@ -1,4 +1,4 @@
-<div class="footer">
+<div class="footer" role="contentinfo">
 <div class="siteinfo">
 <a href="<?php echo $yellow->page->base."/" ?>">&copy; 2018 <?php echo $yellow->page->getHtml("sitename") ?></a>.
 <a href="<?php echo $yellow->text->get("yellowUrl") ?>">Made with Datenstrom Yellow</a>.
diff --git a/system/themes/snippets/header.php b/system/themes/snippets/header.php
index aeb3692..9205c29 100644
--- a/system/themes/snippets/header.php
+++ b/system/themes/snippets/header.php
@@ -16,7 +16,7 @@
 <?php if($page = $yellow->pages->find($yellow->lookup->getDirectoryLocation($yellow->page->location).$yellow->page->get("sidebar"))) $yellow->page->setPage("sidebar", $page) ?>
 <?php if($yellow->page->isPage("sidebar")) $yellow->page->set("pageClass", $yellow->page->get("pageClass")." with-sidebar") ?>
 <div class="<?php echo $yellow->page->getHtml("pageClass") ?>">
-<div class="header">
+<div class="header" role="banner">
 <div class="sitename">
 <h1><a href="<?php echo $yellow->page->base."/" ?>"><i class="sitename-logo"></i><?php echo $yellow->page->getHtml("sitename") ?></a></h1>
 <?php if($yellow->page->isExisting("tagline")): ?><h2><?php echo $yellow->page->getHtml("tagline") ?></h2><?php endif ?>
diff --git a/system/themes/snippets/navigation-sidebar.php b/system/themes/snippets/navigation-sidebar.php
index 16cc9a7..f295061 100644
--- a/system/themes/snippets/navigation-sidebar.php
+++ b/system/themes/snippets/navigation-sidebar.php
@@ -1,6 +1,6 @@
 <?php $pages = $yellow->pages->top() ?>
 <?php $yellow->page->setLastModified($pages->getModified()) ?>
-<div class="navigation">
+<div class="navigation" role="navigation">
 <ul>
 <?php foreach($pages as $page): ?>
 <li><a<?php echo $page->isActive() ? " class=\"active\"" : "" ?> href="<?php echo $page->getLocation(true) ?>"><?php echo $page->getHtml("titleNavigation") ?></a></li>
diff --git a/system/themes/snippets/navigation-tree.php b/system/themes/snippets/navigation-tree.php
index 90bc5c0..030d1aa 100644
--- a/system/themes/snippets/navigation-tree.php
+++ b/system/themes/snippets/navigation-tree.php
@@ -2,7 +2,7 @@
 <?php if(!$pages) $pages = $yellow->pages->top() ?>
 <?php $yellow->page->setLastModified($pages->getModified()) ?>
 <?php if(!$level): ?>
-<div class="navigation-tree">
+<div class="navigation-tree" role="navigation">
 <?php endif ?>
 <ul>
 <?php foreach($pages as $page): ?>
diff --git a/system/themes/snippets/navigation.php b/system/themes/snippets/navigation.php
index 16cc9a7..f295061 100644
--- a/system/themes/snippets/navigation.php
+++ b/system/themes/snippets/navigation.php
@@ -1,6 +1,6 @@
 <?php $pages = $yellow->pages->top() ?>
 <?php $yellow->page->setLastModified($pages->getModified()) ?>
-<div class="navigation">
+<div class="navigation" role="navigation">
 <ul>
 <?php foreach($pages as $page): ?>
 <li><a<?php echo $page->isActive() ? " class=\"active\"" : "" ?> href="<?php echo $page->getLocation(true) ?>"><?php echo $page->getHtml("titleNavigation") ?></a></li>
diff --git a/system/themes/snippets/pagination.php b/system/themes/snippets/pagination.php
index 3a7cf4c..87769be 100644
--- a/system/themes/snippets/pagination.php
+++ b/system/themes/snippets/pagination.php
@@ -1,6 +1,6 @@
 <?php list($name, $pages) = $yellow->getSnippetArgs() ?>
 <?php if($pages->isPagination()): ?>
-<div class="pagination">
+<div class="pagination" role="navigation">
 <?php if($pages->getPaginationPrevious()): ?>
 <a class="previous" href="<?php echo $pages->getPaginationPrevious() ?>"><?php echo $yellow->text->getHtml("paginationPrevious") ?></a>
 <?php endif ?>
diff --git a/system/themes/snippets/sidebar.php b/system/themes/snippets/sidebar.php
index de26068..7a7de30 100644
--- a/system/themes/snippets/sidebar.php
+++ b/system/themes/snippets/sidebar.php
@@ -3,7 +3,7 @@
 <?php $page = $yellow->page->getParentTop(true) ?>
 <?php $pages = $page->getChildren(!$page->isVisible()) ?>
 <?php $yellow->page->setLastModified($pages->getModified()) ?>
-<div class="sidebar">
+<div class="sidebar" role="complementary">
 <div class="navigation-sidebar">
 <p><?php echo $page->getHtml("titleNavigation") ?></p>
 <ul>
@@ -17,7 +17,7 @@
 <?php $page = $yellow->page->getPage("sidebar") ?>
 <?php $page->setPage("main", $yellow->page) ?>
 <?php $yellow->page->setLastModified($page->getModified()) ?>
-<div class="sidebar">
+<div class="sidebar" role="complementary">
 <?php echo $page->getContent() ?>
 </div>
 <?php endif ?>