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

Probleme dans l'utilisation du Header #286

Closed
martinratinaud opened this issue Jul 18, 2024 · 6 comments
Closed

Probleme dans l'utilisation du Header #286

martinratinaud opened this issue Jul 18, 2024 · 6 comments

Comments

@martinratinaud
Copy link

Bonjour à tous

j'ai ce message d'erreur

dsfr : HeaderLinks # -  header__tools-links content is different from header__menu-links content.As you're using a dynamic framework, you should handle duplication of this content yourself, please refer to documentation:https://www.systeme-de-design.gouv.fr/elements-d-interface/composants/en-tete 

dans ce fichier (nous sommes en pleine migration vers react-dsfr

J'ai eu beau cherché dans la doc, je n'ai pas compris comment résoudre ce problème

Merci pour votre aide

@garronej
Copy link
Collaborator

Salut @martinratinaud,

Je crois que c'est un problème interne du DSFR, il faut que je regarde.
@revolunet @enguerranws on a déjà vu çe problème non? Qu'es que vous pensez qu'il faut faire?

@totakoko
Copy link
Contributor

Le warning est côté DSFR, quand toolsHtmlDuplicateId != menuHtml.
Voici les valeurs pour notre cas :

// toolsHtmlDuplicateId = 
<ul class="fr-btns-group"><li><a id="fr-header-quick-access-item--Espacegestionnaire-0-mobile" class="fr-btn fr-icon-account-circle-line" href="/connexion">Espace gestionnaire</a></li></ul>
// menuHtml = 
<ul class="fr-btns-group"><li><a id="fr-header-quick-access-item--Espacegestionnaire_mobile-0" class="fr-btn fr-icon-account-circle-line" href="/connexion">Espace gestionnaire</a></li></ul>

Donc on a un suffixe d'id qui vaut -0-mobile et sinon _mobile-0, donc ça ne doit pas être grand chose à corriger j'imagine. 🕵️

@garronej
Copy link
Collaborator

Lovely! oui je devrais pouvoir corriger ça! Merci pour le rapport détailé.

@garronej
Copy link
Collaborator

Je m'occupe de ça

garronej added a commit that referenced this issue Jul 22, 2024
@garronej
Copy link
Collaborator

garronej commented Jul 22, 2024

Fixed thanks for reporting!

garronej added a commit that referenced this issue Aug 8, 2024
@garronej
Copy link
Collaborator

garronej commented Aug 8, 2024

Actually fixed now

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

No branches or pull requests

3 participants