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

fix: Overflow du texte sur la modale principale #1172

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

Naorid
Copy link
Contributor

@Naorid Naorid commented Dec 12, 2023

En mobile, lorsque le message sur la premiere modale est trop long, il dépasse du navigateur et il est impossible de scroll pour voir l'entièreté du texte.

@AmauriC
Copy link
Owner

AmauriC commented Dec 12, 2023

Merci d'aborder le sujet 😉

En effet ce n'est pas top quand le texte est long mais l'ajout d'un scroll n'est pas idéal non plus.

Réduire la taille du texte avec une règle css ne réglerai pas le problème dans ton cas ?

´@media only screen and (max-width: 600px) {}´

@Naorid
Copy link
Contributor Author

Naorid commented Dec 13, 2023

Merci pour la réactivité :)

Ajouter une règle css permettrait de resoudre le pb pour des texte un peu plus long que celui par défaut.
Mais dans un cas ou on doit afficher un texte très long (type politique de confidentialité), le scroll me semble pertinent

@AmauriC
Copy link
Owner

AmauriC commented Dec 13, 2023

D'acc je comprend. Possible d'avoir l'url du site ou le contenu du texte pour que je puisse faire des tests de mon côté?

@Naorid
Copy link
Contributor Author

Naorid commented Dec 13, 2023

Yep, ca fait suite à une demande de notre responsable RGPD qui tient à que le disclaimer soit présenté dès l'arrivé sur le site :
La Délégation générale à l’emploi et à la formation professionnelle (DGEFP), utilise des cookies de mesure d’audience pour comprendre le parcours de navigation afin d’optimiser l’ergonomie. Vous avez cependant la possibilité de vous opposer à leur usage. En cas de refus, vous ne participerez pas à notre système de mesure d’audience, et votre navigation ne sera pas affectée.

Certaines fonctionnalités du site (lecteurs vidéos, campagne de communication) font par ailleurs appel à des services proposés par des tiers. Ces fonctionnalités déposent des cookies permettant aux tiers d’identifier les sites que vous consultez et les contenus auxquels vous vous intéressez. Ces types de cookies ne sont déposés que si vous donnez votre accord.

Vous pouvez ci-dessous vous informer sur la nature de chacun des cookies déposés, les accepter en cliquant sur “Accepter” ou les refuser en cliquant sur “Refuser”. Ce choix est possible soit en cliquant sur « Personnaliser » ou à tout moment de votre navigation sur le site en cliquant sur « Gérer mes cookies » en bas à gauche.

Pour en savoir plus, rendez-vous sur la politique de confidentialité de 1 jeune 1 solution.

@AmauriC
Copy link
Owner

AmauriC commented Dec 13, 2023

Merci 👌

En alliant les 2, le résultat me semble satisfait pour des longs et très long texte:

@media only screen and (max-width: 768px) {
  #tarteaucitronRoot span#tarteaucitronDisclaimerAlert {
      font-size: 16px;
  }
}

span#tarteaucitronDisclaimerAlert {
    overflow: scroll;
    max-height: 50vh;
}

Qu'est ce que tu en penses?

@Naorid Naorid force-pushed the fix/css-text-overflow-modal branch from 10f75ac to 56b6775 Compare December 14, 2023 14:47
@Naorid
Copy link
Contributor Author

Naorid commented Dec 14, 2023

Nickel
J'ai modifié le commit du coup 👍

@AmauriC
Copy link
Owner

AmauriC commented Dec 29, 2023

Parfait, merci : )
Je m'occupe d'approuver et mettre en prod la semaine prochaine 👌

Copy link

@Videoxasoon Videoxasoon left a comment

Choose a reason for hiding this comment

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

@AmauriC AmauriC merged commit 0485a35 into AmauriC:master Jan 17, 2024
2 checks passed
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.

3 participants