Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FO - Formulaire - Accessibilité - 9.X] Vérifier structure du document #2587

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented May 24, 2024

Ticket

#2544

Description

[Critère 9.1] : Dans chaque page web, l'information est-elle structurée par l'utilisation appropriée de titres ? ---> hiérarchie de titres revue
[Critère 9.2]: Dans chaque page web, la structure du document est-elle cohérente (hors cas particuliers) ? ---> balise main ajoutée
[Critère 9.3]: Dans chaque page web, chaque liste est-elle correctement structurée ? ---> RAS toutes les listes sont bien structurées

Changements apportés

  • Ajout d'une balise main dans le twig (et tant qu'à faire pour les stats aussi et retrait du h4 dans le footer qui causait une erreur
  • Revue de la hiérarchie de titre générale (le h1 passe sur "Mon signalement" car on n'a pas de titre à chaque écran, et les h2 ne sont pas obligatoires)
  • Revue de la hiérarchie de titres sur des pages en particulier (les récap notamment)

Pré-requis

npm run build

Pour utiliser https://xi.github.io/a11y-outline/ il faut commenter la variable d'env SECURITY_CSP_HEADER_VALUE

On peut aussi utiliser l'extension Axe DevTools (ou un équivalent)

Tests

  • Parcourir tous les écrans et vérifier la hiérarchie des titres sur chaque écran

@hmeneuvrier hmeneuvrier changed the title [FO - Formulaire - Accessibilité - 9.X] Vérifier structure du documen [FO - Formulaire - Accessibilité - 9.X] Vérifier structure du document May 24, 2024
Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deux commentaires de relecture ; pas encore testé !

@hmeneuvrier hmeneuvrier force-pushed the feature/2544-fo-formulaire-accessibilite-9x-structure-document branch from e88bd99 to 9640268 Compare May 30, 2024 11:27
Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quelques remarques :
Je pense que le texte de la partie adresse et coordonnées devrait être une liste
adresse
On pourrait ajouter les roles list et listitem pour les bloc de sélection de désordres
role_list
Ce n'est que mon avis mais l'espacement au dessus des titre principaux devrait être réduit (il était déjà assez élevé, et il a encore grandi)
espacement

@hmeneuvrier hmeneuvrier force-pushed the feature/2544-fo-formulaire-accessibilite-9x-structure-document branch from 9640268 to 1f4b38e Compare May 30, 2024 12:49
@hmeneuvrier
Copy link
Collaborator Author

Quelques remarques :

@numew
j'ai traité les 2 premières remarques, pour la 3è c'est dans la PR #2614

Copy link

sonarcloud bot commented May 30, 2024

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK lecture et tests !

@emilschn emilschn merged commit f0368b8 into develop May 31, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the feature/2544-fo-formulaire-accessibilite-9x-structure-document branch June 11, 2024 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants