From ab0285b0aa05574ae864bef590ffd6ea36f0c50a Mon Sep 17 00:00:00 2001
From: maTh <math-home@web.de>
Date: Wed, 21 Jul 2021 22:42:04 +0200
Subject: Add HTML5 tags (#3651)

* use HTML5 tags #3643

added some HTML5 tags: header, main, nav

* <main> into <div>

as dicussed in the PR.
Todo: check side effects (f.e. threepaneview extension)

* fixed whitespace with tabs

* fixed more whitespaces with tabs

it was not my fault, but I fixed it

* added empty lines as wished

* Update app/views/index/global.phtml

Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>

* Update app/views/index/global.phtml

* Update app/views/index/global.phtml

* Update app/views/index/global.phtml

* Update app/views/index/global.phtml

* Update app/views/index/global.phtml

* Update app/views/index/global.phtml

Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
Co-authored-by: Frans de Jonge <fransdejonge@gmail.com>
---
 app/layout/aside_feed.phtml  |  4 ++--
 app/layout/header.phtml      |  8 ++++----
 app/layout/nav_entries.phtml | 12 +++++++-----
 app/layout/nav_menu.phtml    |  4 ++--
 4 files changed, 15 insertions(+), 13 deletions(-)

(limited to 'app/layout')

diff --git a/app/layout/aside_feed.phtml b/app/layout/aside_feed.phtml
index c3f066601..51f138e79 100644
--- a/app/layout/aside_feed.phtml
+++ b/app/layout/aside_feed.phtml
@@ -10,7 +10,7 @@
 	$state_filter = FreshRSS_Context::$user_conf->show_fav_unread ? '&state=3' : '';
 ?>
 
-<div class="aside aside_feed<?= $class ?>" id="aside_feed">
+<nav class="aside aside_feed<?= $class ?>" id="aside_feed">
 	<a class="toggle_aside" href="#close"><?= _i('close') ?></a>
 
 	<?php if (FreshRSS_Auth::hasAccess()) { ?>
@@ -111,7 +111,7 @@
 		<li class="tree-bottom"></li>
 	</ul>
 	</form>
-</div>
+</nav>
 
 <div id="first_load" class="loading"></div>
 <?php flush(); ?>
diff --git a/app/layout/header.phtml b/app/layout/header.phtml
index 7603cf2e5..cf7e4d20e 100644
--- a/app/layout/header.phtml
+++ b/app/layout/header.phtml
@@ -12,7 +12,7 @@ if (FreshRSS_Auth::accessNeedsAction()) {
 }
 ?>
 
-<div class="header">
+<header class="header">
 	<div class="item title">
 		<h1>
 			<a href="<?= _url('index', 'index') ?>">
@@ -51,7 +51,7 @@ if (FreshRSS_Auth::accessNeedsAction()) {
 	</div>
 
 	<?php if (FreshRSS_Auth::hasAccess()) { ?>
-	<div class="item configure">
+	<nav class="item configure">
 		<div class="dropdown">
 			<div id="dropdown-configure" class="dropdown-target"></div>
 			<a class="btn dropdown-toggle" href="#dropdown-configure"><?= _i('configure') ?></a>
@@ -96,10 +96,10 @@ if (FreshRSS_Auth::accessNeedsAction()) {
 				<?php endif; ?>
 			</ul>
 		</div>
-	</div>
+	</nav>
 	<?php } elseif (FreshRSS_Auth::accessNeedsAction()) { ?>
 	<div class="item configure">
 		<?= _i('login') ?><a class="signin" href="<?= _url('auth', 'login') ?>"><?= _t('gen.auth.login') ?></a>
 	</div>
 	<?php } ?>
-</div>
+</header>
diff --git a/app/layout/nav_entries.phtml b/app/layout/nav_entries.phtml
index cbc514737..55e1743dc 100644
--- a/app/layout/nav_entries.phtml
+++ b/app/layout/nav_entries.phtml
@@ -1,5 +1,7 @@
-<ul id="nav_entries">
-	<li class="item"><a class="previous_entry" href="#"><?= _i('prev') ?></a></li>
-	<li class="item"><a class="up" href="#"><?= _i('up') ?></a></li>
-	<li class="item"><a class="next_entry" href="#"><?= _i('next') ?></a></li>
-</ul>
\ No newline at end of file
+<nav>
+	<ul id="nav_entries">
+		<li class="item"><a class="previous_entry" href="#"><?= _i('prev') ?></a></li>
+		<li class="item"><a class="up" href="#"><?= _i('up') ?></a></li>
+		<li class="item"><a class="next_entry" href="#"><?= _i('next') ?></a></li>
+	</ul>
+</nav>
diff --git a/app/layout/nav_menu.phtml b/app/layout/nav_menu.phtml
index 0724abc16..874a4273a 100644
--- a/app/layout/nav_menu.phtml
+++ b/app/layout/nav_menu.phtml
@@ -2,7 +2,7 @@
 	$actual_view = Minz_Request::actionName();
 ?>
 
-<div class="nav_menu">
+<nav class="nav_menu">
 	<?php if ($actual_view === 'normal' || $actual_view === 'reader') { ?>
 	<a class="btn toggle_aside" href="#aside_feed"><?= _i('category') ?></a>
 	<?php } ?>
@@ -225,5 +225,5 @@
 	<?php if (FreshRSS_Auth::hasAccess() || FreshRSS_Context::$system_conf->allow_anonymous_refresh) { ?>
 	<a id="actualize" class="btn" href="<?= _url('feed', 'actualize') ?>" title="<?= _t('gen.action.actualize') ?>"><?= _i('refresh') ?></a>
 	<?php } ?>
-</div>
+</nav>
 <?php flush(); ?>
-- 
cgit v1.2.3