Skip to content

Commit f32052a

Browse files
committed
fix(a11y): titres de page, cas d'usages — liens, liste, tableau (NC 5.3, 6.1, 8.5, 9.3, 13.2)
Titres de page (NC 8.5): - home AE/AP, accessibility, mentions, errors, profile, changelogs: content_for :title Cas d'usages: - _others: div.use_cases → ul.fr-raw-list + li (NC 9.3) + aria-label contextuel sur les liens "En savoir plus" (NC 6.1) - _simplifions_card: fr-sr-only "(ouverture dans un nouvel onglet)" (NC 13.2) - _endpoints_table: <caption class="fr-sr-only"> (NC 5.3) - index AE/AP: content_for :title (NC 8.5) Partially solves API-6952
1 parent 2c47b86 commit f32052a

12 files changed

Lines changed: 20 additions & 6 deletions

File tree

site/app/views/api_entreprise/cas_usages/index.html.erb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
<% content_for :title, t('.title') %>
12
<div id="cas-usages" class="fr-container--fluid">
23
<div class="banner fr-pt-5w fr-pb-8w">
34
<div class="fr-container">

site/app/views/api_entreprise/pages/home.html.erb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
<% content_for :title, 'Accueil' %>
12
<div id="homepage">
23
<%= render 'shared/pages/home_alerts' %>
34
<%= render 'api_entreprise/pages/home/section_welcome' %>

site/app/views/api_particulier/cas_usages/index.html.erb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
<% content_for :title, t('.title') %>
12
<div id="cas-usages" class="fr-container--fluid">
23
<div class="banner fr-pt-5w fr-pb-8w">
34
<div class="fr-container">

site/app/views/api_particulier/pages/home.html.erb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
<% content_for :title, 'Accueil' %>
12
<div id="homepage">
23
<%= render 'shared/pages/home_alerts' %>
34
<%= render 'api_particulier/pages/home/section_welcome' %>

site/app/views/errors/show.html.erb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
<% content_for :title, t(".#{@rescue_response}.title", default: 'Erreur') rescue content_for :title, 'Erreur' %>
12
<div class="fr-container fr-mb-12w fr-mt-12w center">
23
<h1 class="text-xl">
34
<%= t(".#{@rescue_response}.title") %>

site/app/views/shared/cas_usages/_endpoints_table.html.erb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
<div class="fr-table">
22
<table>
3+
<caption class="fr-sr-only">API disponibles pour ce cas d'usage</caption>
34
<thead>
45
<tr>
56
<th scope="col"><%= t('.headers.api') %></th>
Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,15 @@
11
<h3><%= t('.title').html_safe %></h3>
2-
<div class="use_cases">
2+
<ul class="use_cases fr-raw-list">
33
<% @other_fiches_pratiques.sample(3).each do |use_case| %>
4-
<div class="use_case fr-mt-4v">
4+
<li class="use_case fr-mt-4v">
55
<%= image_tag("#{api}/cas_usages/pictos/#{use_case.uid}.svg", alt: "") %>
66
<h4><%= t(".#{use_case.uid}.title") %></h4>
77
<p><%= t(".#{use_case.uid}.description") %></p>
8-
<%= link_to t('.link'), cas_usage_path(use_case.uid), class: "fr-btn fr-btn--secondary" %>
9-
</div>
8+
<%= link_to cas_usage_path(use_case.uid),
9+
class: "fr-btn fr-btn--secondary",
10+
aria: { label: "#{t('.link')}#{t(".#{use_case.uid}.title")}" } do %>
11+
<%= t('.link') %>
12+
<% end %>
13+
</li>
1014
<% end %>
11-
</div>
15+
</ul>

site/app/views/shared/cas_usages/_simplifions_card.html.erb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
<div class="fr-card__content">
44
<h3 class="fr-card__title fr-h5">
55
<%= link_to use_case.url, target: '_blank', rel: 'noopener noreferrer' do %>
6-
<% if use_case.icon.present? %><span aria-hidden="true"><%= use_case.icon %></span> <% end %><%= use_case.name %>
6+
<% if use_case.icon.present? %><span aria-hidden="true"><%= use_case.icon %></span> <% end %><%= use_case.name %><span class="fr-sr-only"> (ouverture dans un nouvel onglet)</span>
77
<% end %>
88
</h3>
99
<p class="fr-card__desc"><%= use_case.description %></p>

site/app/views/shared/changelogs/index.html.erb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
<% content_for :title, t('.title') %>
12
<div class="fr-grid-row">
23
<div class="fr-col-12 fr-col-md-9 fr-col-lg-8">
34
<h1 class="fr-mb-2w">

site/app/views/shared/pages/accessibility.html.erb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
<% content_for :title, 'Accessibilité' %>
12
<div class="fr-grid-row">
23
<div class="fr-col-12 fr-col-sm-8 fr-col-offset-sm-2">
34
<h1 class="center">

0 commit comments

Comments
 (0)