<Display> utilisé dans <Header> #345
Comments
|
Bonjour @martinratinaud, Il y a un certain nombre de remarques ici qui ne sont pas spécifiquement liées à react-dsfr, mais qui concernent le DSFR dans son ensemble. Ce serait peut-être pertinent de faire remonter ces points au SIG : https://github.com/GouvernementFR/dsfr.
Une option pourrait être ajoutée pour désactiver le composant Display, si cela pose des problèmes pour le SEO. Cela dit, je comprends qu'il puisse y avoir un enjeu en termes d'opportunité pour proposer un mode sombre dans certains cas mais le DSFR propose des composants et un système de couleurs qui sont conçus pour permettre l'intégration d'un mode sombre sans coût de développement supplémentaire. Je ne suis pas certain de comprendre pourquoi cette possibilité devrais être désactivée. De manière plus personnelle, je trouve que les sites sans mode sombre peuvent être un peu difficiles à lire, et j'essaie de passer le moins de temps possible dessus. Il est possible que je ne sois pas le seul dans cette situation, et ne pas proposer de mode sombre pourrait avoir un impact sur votre rétention. |
|
Bonjour @garronej et merci pour la réponse La prise en compte d'un mode sombre, au delà du DSFR, est couteuse, meme si les composants du DSFR le gèrent bien. En effet, sur une codebase existante avec un vécu de 3 ans, une intégration incrémentale du DSFR doit pouvoir être possible et cela passe par choisir si un mode sombre est envisageable ou non. C'est d'ailleurs en passant au DSFR que nous nous sommes rendus compte de tous les impacts visuels qu'il y avait (sur les cartes mapview et plein d'autres) et que nos utilisateurs et donneurs d'ordre nous ont fait remarquer que certains textes n'étaient aps visible (noir sur noir ou blanc sur blanc). Le cout pour tout mettre à jour et maintenir ensuite la code base a été rejeté, nous avons donc supprimé le mode sombre. Merci |
…nt de theme sombre/clair Voir codegouvfr/react-dsfr#345
…nt de theme sombre/clair Voir codegouvfr/react-dsfr#345
|
Oui d’accord, je comprends que dans le cas d’une migration incrémentale, cela peut être un peu compliqué. Je vais essayer de faire en sorte que le composant display ne soit pas systématiquement inclus. |
|
Hello ! Qu'en pensez vous ? |
|
J'ajoute que cela concerne aussi les modales qui pourrait être concernés. En fait, il y a toujours un clash entre le SEO et l'accessibilité. On essaie de faire avec les deux mondes mais l'astuce du div en role heading est une très bonne alternative pour concilier les deux mondes. Il faudrait pouvoir l'intégrer à la demande sur une modale. Surtout que l'utilisation du H1 dans la modale sur DSFR reste à discuter d'après leur doc. Dans la doc, il parle de contenu annexe qui pourrait être ouvert dans une autre page. Dans le cas des paramètres d'affichage, je ne pense pas que l'on rentre dans ce cas de figure. |
|
En attente d'un retour GouvernementFR/dsfr#1062 |
|
Bonjour, Une réponse à été faite dans le ticket GouvernementFR/dsfr#1062 |
|
Thanks for the answer @keryanS! @ddecrulle so I think we should maybe consider mounting the display component in the footer so it's at the bottom of the page. Or use some portals |
Bonjour, sur FCU 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
Une possible solution serait
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
The text was updated successfully, but these errors were encountered: