From e637f3e426a2f2cfeb0c0aa8c49e9c41b3098317 Mon Sep 17 00:00:00 2001
From: Chocobozzz <me@florianbigard.com>
Date: Thu, 26 Sep 2024 15:48:15 +0200
Subject: [PATCH] Fix admin modal accessibility

---
 .../src/app/modal/admin-welcome-modal.component.html | 12 ++++++------
 .../src/app/modal/admin-welcome-modal.component.scss | 11 +++--------
 2 files changed, 9 insertions(+), 14 deletions(-)

diff --git a/client/src/app/modal/admin-welcome-modal.component.html b/client/src/app/modal/admin-welcome-modal.component.html
index d3302b53f..9179d062a 100644
--- a/client/src/app/modal/admin-welcome-modal.component.html
+++ b/client/src/app/modal/admin-welcome-modal.component.html
@@ -10,20 +10,20 @@
 
     <div class="block-documentation">
       <div class="columns">
-        <a class="link-block" href="https://docs.joinpeertube.org/maintain/tools" target="_blank" rel="noopener noreferrer">
-          <a i18n class="link-title" href="https://docs.joinpeertube.org/maintain/tools" target="_blank" rel="noopener noreferrer">CLI documentation</a>
+        <a class="link-block" href="https://docs.joinpeertube.org/maintain/tools" target="_blank" i18n-title title="Go to the CLI documentation">
+          <h5 i18n class="underline-orange">CLI documentation</h5>
 
           <div i18n>Upload or import videos, parse logs, prune storage directories, reset user password...</div>
         </a>
 
-        <a class="link-block" href="https://docs.joinpeertube.org/admin/following-instances" target="_blank" rel="noopener noreferrer">
-          <a i18n class="link-title" href="https://docs.joinpeertube.org/admin/following-instances" target="_blank" rel="noopener noreferrer">Administer documentation</a>
+        <a class="link-block" href="https://docs.joinpeertube.org/admin/following-instances" target="_blank" i18n-title title="Go to the admin documentation">
+          <h5 i18n class="underline-orange">Admin documentation</h5>
 
           <div i18n>Managing users, following other instances, dealing with spammers...</div>
         </a>
 
-        <a class="link-block" href="https://docs.joinpeertube.org/use/setup-account" target="_blank" rel="noopener noreferrer">
-          <a i18n class="link-title" href="https://docs.joinpeertube.org/use/setup-account" target="_blank" rel="noopener noreferrer">Use documentation</a>
+        <a class="link-block" href="https://docs.joinpeertube.org/use/setup-account" target="_blank" i18n-title title="Go to the user documentation">
+          <h5 i18n class="underline-orange">User documentation</h5>
 
           <div i18n>Setup your account, managing video playlists, discover third-party applications...</div>
         </a>
diff --git a/client/src/app/modal/admin-welcome-modal.component.scss b/client/src/app/modal/admin-welcome-modal.component.scss
index fec5e078d..c9b62658c 100644
--- a/client/src/app/modal/admin-welcome-modal.component.scss
+++ b/client/src/app/modal/admin-welcome-modal.component.scss
@@ -50,6 +50,7 @@ li {
   padding: 10px;
   transition: background-color 0.2s ease-in;
   flex-basis: 33%;
+  text-align: center;
 
   @include disable-default-a-behaviour;
 
@@ -57,15 +58,9 @@ li {
     background-color: rgba(0, 0, 0, 0.05);
   }
 
-  .link-title {
+  > h5 {
     font-weight: $font-semibold;
-    display: flex;
-    justify-content: center;
+    font-size: 1rem;
     margin-bottom: 5px;
   }
-
-  .link-title,
-  div {
-    text-align: center;
-  }
 }