8000 iconId obligatoire dans le header > quickAccessItems · Issue #374 · codegouvfr/react-dsfr · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

iconId obligatoire dans le header > quickAccessItems #374

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

Open
Chanizo opened this issue Feb 6, 2025 · 3 comments
Open

iconId obligatoire dans le header > quickAccessItems #374

Chanizo opened this issue Feb 6, 2025 · 3 comments
Assignees

Comments

@Chanizo
Copy link
Chanizo commented Feb 6, 2025

Bonjour,

J'ai une question concernant les quickAccessItems dans le composant Header, nous avons l'un de nos quickAccessItem qui redirige en fait vers un lien externe, typiquement c'est une FAQ qui est mutualisée avec d'autres apps.

Nous avons donc l'icone de lien externe (automatique) et obligatoire d'après le dsfr qui apparait, et l'on se retrouve avec un iconId obligatoire (props du component) sur l'item en plus, donc une deuxième icone pour le même item...

Nous avons essayé de regarder côté dsfr, on a l'impression que l'icone n'est pas forcément obligatoire, est-ce possible de la rendre optionnelle sur les quickAccessItems ?

Merci d'avance

@garronej
Copy link
Collaborator
garronej commented Feb 6, 2025

Bonjour @Chanizo,

Oui, sentez vous libre d'appliquer du CSS custom pour ce genre de situation:

https://react-dsfr.codegouv.studio/components
https://react-dsfr.codegouv.studio/css-in-js

Si vous avez des difficultées et que votre repo est publique, je peut regarder

@Chanizo
Copy link
Author
Chanizo commented Feb 6, 2025

Merci pour votre réponse et la rapidité,

Pour être sûr de bien comprendre, vous proposez d'écraser le style pour l'icône automatique de lien externe ?
Si c'est bien ça, très bien ça nous permettra de la faire disparaitre

Mais en fait, on aurait aimé que les icones soient optionnelles au niveau des quickAccessItems du Header, est-ce possible ? (le champ iconId est obligatoire d'après le typage...)
On garderait seulement l'icone de lien externe du coup, sur le lien en question

@enguerranws
Copy link
Collaborator

J'ai posé la question côté DSFR pour savoir ce qu'on fait à ce sujet. Le vrai souci, c'est que de leur côté, ils ne précisent pas de règle pour ce cas (et le DOM associé) : dans les exemples d'utilisations, il y a toujours un icône associé à un quickAccessItem, donc on l'a transposé tel quel (obligatoire) comme ça côté react-dsfr.

@enguerranws enguerranws self-assigned this Mar 3, 2025
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
0