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

Ajustement de l'icône des liens externes dans les textes #526

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Olivia206
Copy link
Contributor

@Olivia206 Olivia206 commented Jul 25, 2024

Type

  • Nouvelle fonctionnalité
  • Bug
  • Ajustement
  • Rangement

Description

Avec les nouvelles icônes, moins délicates que les précédentes, on a un rendu sur les liens externe un peu brouillon et chargé :

Capture d’écran 2024-07-25 à 17 08 33

J'ai ajouté un display: inline-block à l'icône pour qu'elle compte dans le lien sans être soulignée.

Niveau d'incidence

  • Incidence faible 😌
  • Incidence moyenne 😲
  • Incidence forte 😱

URL de test sur example.osuny.org

http://localhost:1313/fr/blocks/blocs-de-mise-en-page/

Screenshots

Capture d’écran 2024-07-25 à 17 08 26
Capture d’écran 2024-07-26 à 10 48 48

Dans un crédit :

Capture d’écran 2024-07-26 à 10 48 55

@alexisben
Copy link
Contributor

Bon le seul hic c'est que du coup le picto peut chasser à la ligne... À ce jour on a pas trouvé de recette magique pour éviter le soulignement et empêcher le picto de chasser tout en restant dans le flux inline.

image

@Olivia206
Copy link
Contributor Author

Olivia206 commented Jul 29, 2024

Bon le seul hic c'est que du coup le picto peut chasser à la ligne... À ce jour on a pas trouvé de recette magique pour éviter le soulignement et empêcher le picto de chasser tout en restant dans le flux inline.

Ah oui mince 😞

J'ai fait le test avec un lien contenant un span, et seul le span a le souligné (en ignorant ma PR, juste avec le style initial), tu en penses quoi ?

Capture d’écran 2024-07-29 à 12 19 07

@Olivia206 Olivia206 marked this pull request as draft September 6, 2024 13:30
@alexisben
Copy link
Contributor

Bon le seul hic c'est que du coup le picto peut chasser à la ligne... À ce jour on a pas trouvé de recette magique pour éviter le soulignement et empêcher le picto de chasser tout en restant dans le flux inline.

Ah oui mince 😞

J'ai fait le test avec un lien contenant un span, et seul le span a le souligné (en ignorant ma PR, juste avec le style initial), tu en penses quoi ?

Capture d’écran 2024-07-29 à 12 19 07

J'ai pas bien compris ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adjustment Incidence faible 😌 Risque faible de régression sur les sites
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants