aboutsummaryrefslogtreecommitdiffstats
path: root/core/templates
diff options
context:
space:
mode:
authorGrigorii K. Shartsev <me@shgk.me>2023-10-18 17:02:13 +0200
committerGrigorii K. Shartsev <me@shgk.me>2023-10-21 17:35:44 +0200
commit0e8abfd24c4dc4a3179a31eabbcb7b7eb6bbd598 (patch)
tree331aa46c43c494a9ee8c1ae62004d669a7735f3e /core/templates
parent5c2610af7dbce72fa1c1b02185a2c28cd5be004e (diff)
downloadnextcloud-server-0e8abfd24c4dc4a3179a31eabbcb7b7eb6bbd598.tar.gz
nextcloud-server-0e8abfd24c4dc4a3179a31eabbcb7b7eb6bbd598.zip
fix(core): remove unnecessary role=banner on top level header
By default, the HTML's `<header>` element has an identical meaning to the banner landmark, unless it is a descendant of `<aside>`, `<article>`, `<main>`, `<nav>`, or `<section>`. Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
Diffstat (limited to 'core/templates')
-rw-r--r--core/templates/layout.guest.php2
-rw-r--r--core/templates/layout.user.php2
2 files changed, 2 insertions, 2 deletions
diff --git a/core/templates/layout.guest.php b/core/templates/layout.guest.php
index 7a741d33a74..e90476291bf 100644
--- a/core/templates/layout.guest.php
+++ b/core/templates/layout.guest.php
@@ -33,7 +33,7 @@ p($theme->getTitle());
<div class="wrapper">
<div class="v-align">
<?php if ($_['bodyid'] === 'body-login'): ?>
- <header role="banner">
+ <header>
<div id="header">
<div class="logo"></div>
</div>
diff --git a/core/templates/layout.user.php b/core/templates/layout.user.php
index aebcda5c849..9c23930f324 100644
--- a/core/templates/layout.user.php
+++ b/core/templates/layout.user.php
@@ -56,7 +56,7 @@ p($theme->getTitle());
<?php if ($_['id-app-navigation'] !== null) { ?><a href="<?php p($_['id-app-navigation']); ?>" class="button primary skip-navigation"><?php p($l->t('Skip to navigation of app')); ?></a><?php } ?>
</div>
- <header role="banner" id="header">
+ <header id="header">
<div class="header-left">
<a href="<?php print_unescaped($_['logoUrl'] ?: link_to('', 'index.php')); ?>"
aria-label="<?php p($l->t('Go to %s', [$_['logoUrl'] ?: $_['defaultAppName']])); ?>"