From 4b40638d00641e01a080acec1af0eb49f020b149 Mon Sep 17 00:00:00 2001
From: Eduardo Morales <emoral435@gmail.com>
Date: Mon, 5 Feb 2024 17:11:49 -0600
Subject: [PATCH] fix: added last two places for aria-labels on setting icons

Signed-off-by: Eduardo Morales <emoral435@gmail.com>
---
 .../settings/templates/settings/admin/additional-mail.php | 8 +++++---
 apps/settings/templates/settings/admin/overview.php       | 7 ++++++-
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/apps/settings/templates/settings/admin/additional-mail.php b/apps/settings/templates/settings/admin/additional-mail.php
index 7b88e99736954..3e8b373c73815 100644
--- a/apps/settings/templates/settings/admin/additional-mail.php
+++ b/apps/settings/templates/settings/admin/additional-mail.php
@@ -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>
diff --git a/apps/settings/templates/settings/admin/overview.php b/apps/settings/templates/settings/admin/overview.php
index db066fc6df31f..cb17f76e0d85d 100644
--- a/apps/settings/templates/settings/admin/overview.php
+++ b/apps/settings/templates/settings/admin/overview.php
@@ -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>