fix(seo): retrait des h1 dupliqués #6342
Conversation
m-maillot
force-pushed
the
fix/headings
branch
from
abe80ae to
9377021
Compare
| nonce={nonce} | ||
| defaultColorScheme={pathname.includes("/widgets") ? "light" : "system"} | ||
| > | ||
| <DefaultLayout nonce={nonce} defaultColorScheme={"light"}> |
There was a problem hiding this comment.
On repasse en mode light le temps de pouvoir modifier le h1 de la modale pour le SEO
| @@ -0,0 +1,140 @@ | |||
| "use client"; | |||
There was a problem hiding this comment.
Création d'une nouvelle page pour Besoin plus d'informations au lieu de l'afficher dans une popup.
| }, | ||
| text: "Plan du site", | ||
| }, | ||
| headerFooterDisplayItem, |
There was a problem hiding this comment.
On supprime le lien vers la modal pour le dark/light mode
| Faire une suggestion pour améliorer cette page | ||
| </label> | ||
| <textarea | ||
| aria-describedby="input-:r4:-desc-error" |
There was a problem hiding this comment.
Normal, c'était un bug et la monté en version de react-dsfr l'a corrigé.
| /> | ||
| )} | ||
| onSearchButtonClick={onSearchSubmit} | ||
| disableDisplay={true} |
There was a problem hiding this comment.
On désactive la modal pour choisir entre dark et light mode.
| <div | ||
| role="heading" | ||
| aria-level={2} | ||
| className={`${fr.cx("fr-h2")} ${title}`} | ||
| > |
There was a problem hiding this comment.
On bascule le h2 en div avec le role heading.
| <Image | ||
| src="/static/assets/img/srdt.svg" | ||
| alt="Contactez les services de renseignements au droit du travail au 0800 026 080" | ||
| width={350} | ||
| height={100} | ||
| /> |
There was a problem hiding this comment.
d'un point de vue accessibilité, on nous avez dit de faire en sorte de créer nous même cette affichage il me semble.
après tu me diras avec le alt, ça devrait suffir
There was a problem hiding this comment.
Le alt suffit mais il y a un point en effet sur cette page en terme d'accessibilité. Il n'y a pas le focus quand on passe dessus avec le Tab. On le retravaillera après le pré audit :)
There was a problem hiding this comment.
Il y a aussi la couleur qui est trop terne. Il y a un ticket sur le sujet, on le traitera à ce moment.
| }, | ||
| text: "Plan du site", | ||
| }, | ||
| headerFooterDisplayItem, |
packages/code-du-travail-frontend/src/modules/besoin-plus-informations/index.tsx
Outdated
Show resolved
Hide resolved
m-maillot
force-pushed
the
fix/headings
branch
from
9377021 to
4eac061
Compare
m-maillot
temporarily deployed
to
review-auto
GitHub Actions
Inactive
|
🎉 Deployment for commit 4eac061 : Docker images
|
Il ne restera que les h3 sur les titres du footer : codegouvfr/react-dsfr#349