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

Ajoute plus d'info sur l'impact usager #928

Conversation

bellangerq
Copy link
Collaborator

closes #890

@hissalht hissalht temporarily deployed to ara-890-impact-usager-p-lot0ex January 22, 2025 14:05 Inactive
@hissalht hissalht temporarily deployed to ara-890-impact-usager-p-lot0ex January 22, 2025 15:02 Inactive
@bellangerq
Copy link
Collaborator Author

La largeur de la tooltip du DSFR est automatiquement calculée, on a pas trop le contrôle dessus du coup ça coupe un peu les lignes du contenu.

image

@hissalht hissalht temporarily deployed to ara-890-impact-usager-p-lot0ex January 22, 2025 15:24 Inactive
@bellangerq bellangerq marked this pull request as ready for review January 22, 2025 15:24
@hissalht hissalht temporarily deployed to ara-890-impact-usager-p-lot0ex January 22, 2025 15:33 Inactive
@benoitdequick
Copy link
Contributor

benoitdequick commented Jan 22, 2025

Ça passe sous les onglets des fois :(

Et, il y a quand même beaucoup (trop?) de contenu dans ce tooltip, non ?

@benoitdequick
Copy link
Contributor

bon en fait c'est sans doute logique que ça passe sous les onglets.
Mais en zoom le contenu est tellement imposant qu'on ne peut pas tout lire :(

@bellangerq
Copy link
Collaborator Author

Ça passe sous les onglets des fois :(

Et, il y a quand même beaucoup (trop?) de contenu dans ce tooltip, non ?

C'est vrai qu'il y a pas mal de contenu.

Mmmh pour l'histoire des onglets, je dirais que c'est normal ouais vu que ça fait partie du contenu qui est dans les onglets.

@bellangerq bellangerq force-pushed the 890-impact-usager-pas-toujours-clair-et-difficulté-de-correction-questionnée branch from 4bfb0ac to dc79ee1 Compare January 31, 2025 10:51
@hissalht hissalht temporarily deployed to ara-890-impact-usager-p-lot0ex January 31, 2025 10:51 Inactive
@yaaax
Copy link
Collaborator

yaaax commented Jan 31, 2025

Comme @benoitdequick , je pense que la tooltip devrait être visible par dessus tout. Et la bonne nouvelle c’est que je crois que j’ai une solution : utiliser, comme pour les modales <Teleport to="body"> pour la <span class="fr-tooltip> ;)
(j’ai testé rapidement de bouger la span bien au-dessus de tout dans les devtools, ça fonctionne bien)

Copy link
Collaborator

@hissalht hissalht left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

juste un <template #label> oublié, sinon lgtm

@hissalht hissalht temporarily deployed to ara-890-impact-usager-p-lot0ex February 5, 2025 11:18 Inactive
@hissalht hissalht temporarily deployed to ara-890-impact-usager-p-lot0ex February 5, 2025 11:25 Inactive
@bellangerq bellangerq force-pushed the 890-impact-usager-pas-toujours-clair-et-difficulté-de-correction-questionnée branch from 2edb789 to 2b4b500 Compare February 5, 2025 15:07
@hissalht hissalht temporarily deployed to ara-890-impact-usager-p-lot0ex February 5, 2025 15:07 Inactive
@hissalht hissalht temporarily deployed to ara-890-impact-usager-p-lot0ex February 7, 2025 09:11 Inactive
@hissalht hissalht temporarily deployed to ara-890-impact-usager-p-lot0ex February 7, 2025 09:15 Inactive
@bellangerq bellangerq merged commit 66587fe into main Feb 7, 2025
1 check passed
@bellangerq bellangerq deleted the 890-impact-usager-pas-toujours-clair-et-difficulté-de-correction-questionnée branch February 7, 2025 09:17
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

Successfully merging this pull request may close these issues.

“Impact usager” pas toujours clair et “difficulté de correction” questionnée
4 participants