codegouvfr / codegouvfr/react-dsfr

<Display> utilisé dans <Header>

Open
#345 8 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
496
Forks
99
Avg merge
15h 36m
Merged PRs (30d)
4

Description

Bonjour, sur [FCU](https://france-chaleur-urbaine.beta.gouv.fr) nous utilisons le DSFR et après un audit SEO, nous nous sommes rendu compte que le 1er H1 sur notre page d'accueil affiche "Paramètres d'affichages"

Ceci vient de la modale Display qui est intégrée dans le Header

Plusieurs problèmes se posent ici
- nous n'avons pas de mode sombre donc pas besoin de cette modale mais il n'est [pas possible de l'enlever](https://github.com/codegouvfr/react-dsfr/blob/99a594d6d83dc163f49390f343c700097815687b/src/Header/Header.tsx#L197)
- la modale est directement dans le code source de la page alors qu'elle n'est sensée s'afficher au clic

Une possible solution serait
- un flag "withDisplay" qui enlève cette modale

Pour la modale, cela me parait être une discussion plus circonstanciée à mener dans laquelle mon avis serait de ne pas intégrer le code dans le code source étant donné qu'il n'est pas affiché

Merci

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.