Skip to content

Commit

Permalink
fix: added last two places for aria-labels on setting icons
Browse files Browse the repository at this point in the history
Signed-off-by: Eduardo Morales <emoral435@gmail.com>
  • Loading branch information
emoral435 authored and backportbot[bot] committed Feb 6, 2024
1 parent 2f6b641 commit 4b40638
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 4 deletions.
8 changes: 5 additions & 3 deletions apps/settings/templates/settings/admin/additional-mail.php
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,11 @@
<div class="section" id="mail_general_settings">
<form id="mail_general_settings_form" class="mail_settings">
<h2><?php p($l->t('Email server'));?></h2>
<a target="_blank" rel="noreferrer noopener" class="icon-info"
title="<?php p($l->t('Open documentation'));?>"
href="<?php p(link_to_docs('admin-email')); ?>"></a>
<a target="_blank"
rel="noreferrer noopener" class="icon-info"
title="<?php p($l->t('Open documentation'));?>"
href="<?php p(link_to_docs('admin-email')); ?>"
aria-label="<?php p($l->t('Open documentation'));?>"></a>
<p class="settings-hint">
<?php p($l->t('It is important to set up this server to be able to send emails, like for password reset and notifications.')); ?>
</p>
Expand Down
7 changes: 6 additions & 1 deletion apps/settings/templates/settings/admin/overview.php
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,12 @@
<div id="security-warning" class="section">
<div class="security-warning__heading">
<h2><?php p($l->t('Security & setup warnings'));?></h2>
<a target="_blank" rel="noreferrer" class="icon-info" title="<?php p($l->t('Open documentation'));?>" href="<?php p(link_to_docs('admin-warnings')); ?>"></a>
<a target="_blank"
rel="noreferrer"
class="icon-info"
title="<?php p($l->t('Open documentation'));?>"
href="<?php p(link_to_docs('admin-warnings')); ?>"
aria-label="<?php p($l->t('Open documentation')); ?>"></a>
</div>
<p class="settings-hint"><?php p($l->t('It\'s important for the security and performance of your instance that everything is configured correctly. To help you with that we are doing some automatic checks. Please see the linked documentation for more information.'));?></p>

Expand Down

0 comments on commit 4b40638

Please sign in to comment.