diff options
author | Grigorii K. Shartsev <me@shgk.me> | 2023-10-18 17:02:13 +0200 |
---|---|---|
committer | Grigorii K. Shartsev <me@shgk.me> | 2023-10-21 17:35:44 +0200 |
commit | 0e8abfd24c4dc4a3179a31eabbcb7b7eb6bbd598 (patch) | |
tree | 331aa46c43c494a9ee8c1ae62004d669a7735f3e /core/templates | |
parent | 5c2610af7dbce72fa1c1b02185a2c28cd5be004e (diff) | |
download | nextcloud-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.php | 2 | ||||
-rw-r--r-- | core/templates/layout.user.php | 2 |
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']])); ?>" |