Merge #5316
5316: fix missing aria-labels in footer r=Keirua a=Keirua début du cleanup des exclusions d'accessibilité (et bon candidat pour tester si bors fonctionne comme prévu) Co-authored-by: clemkeirua <clement@keiruaprod.fr>
This commit is contained in:
commit
b135b5cfb0
2 changed files with 6 additions and 6 deletions
|
@ -12,9 +12,9 @@
|
|||
= link_to "beta.gouv.fr", "https://beta.gouv.fr", title: "le site de Beta.gouv.fr"
|
||||
%li
|
||||
= link_to "https://numerique.gouv.fr/", title: "DINUM", 'aria-label': 'DINUM' do
|
||||
%span.footer-logo.footer-logo-dinum{ role: 'img' }
|
||||
%span.footer-logo.footer-logo-dinum{ role: 'img', 'aria-label': 'Logo DINUM' }
|
||||
= link_to "https://beta.gouv.fr", title: "le site de Beta.gouv.fr", 'aria-label': 'beta.gouv.fr' do
|
||||
%span.footer-logo.footer-logo-beta-gouv-fr{ role: 'img' }
|
||||
%span.footer-logo.footer-logo-beta-gouv-fr{ role: 'img', 'aria-label': 'Logo beta.gouv.fr' }
|
||||
%span.footer-logo.footer-logo-france{ role: 'img', 'aria-label': 'République Française' }
|
||||
|
||||
%li.footer-column
|
||||
|
|
|
@ -6,12 +6,12 @@ feature 'wcag rules for usager', js: true do
|
|||
context 'pages without the need to be logged in' do
|
||||
scenario 'homepage' do
|
||||
visit root_path
|
||||
expect(page).to be_accessible.excluding ".footer-logo"
|
||||
expect(page).to be_accessible
|
||||
end
|
||||
|
||||
scenario 'sign_up page' do
|
||||
visit new_user_registration_path
|
||||
expect(page).to be_accessible.excluding ".footer-logo"
|
||||
expect(page).to be_accessible
|
||||
end
|
||||
|
||||
scenario 'account confirmation page' do
|
||||
|
@ -28,12 +28,12 @@ feature 'wcag rules for usager', js: true do
|
|||
|
||||
scenario 'sign_in page' do
|
||||
visit new_user_session_path
|
||||
expect(page).to be_accessible.excluding ".footer-logo", '#user_email'
|
||||
expect(page).to be_accessible.excluding '#user_email'
|
||||
end
|
||||
|
||||
scenario 'contact page' do
|
||||
visit contact_path
|
||||
expect(page).to be_accessible.excluding ".footer-logo"
|
||||
expect(page).to be_accessible
|
||||
end
|
||||
|
||||
scenario 'commencer page' do
|
||||
|
|
Loading…
Reference in a new issue