Skip to content

Commit

Permalink
fix(core): remove unnecessary role=contentinfo from page footer on login
Browse files Browse the repository at this point in the history
`<footer>` has this role initially then it is a page footer, not inside `section`/`article`, and etc.

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
  • Loading branch information
ShGKme committed Oct 19, 2023
1 parent 59d2244 commit 9945b2e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion core/css/guest.scss
Expand Up @@ -741,7 +741,7 @@ footer {
font-weight: bold;
}

&[role="contentinfo"] {
&.guest-box {
padding: 6px 24px;
margin-bottom: 1rem;

Expand Down
2 changes: 1 addition & 1 deletion core/templates/layout.guest.php
Expand Up @@ -47,7 +47,7 @@
</main>
</div>
</div>
<footer role="contentinfo" class="guest-box">
<footer class="guest-box">
<p class="info">
<?php print_unescaped($theme->getLongFooter()); ?>
</p>
Expand Down

0 comments on commit 9945b2e

Please sign in to comment.