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

[BUGFIX] Le figcaption des éléments images produisent toujours un espace vide #11602

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

yannbertrand
Copy link
Member

🥞 Problème

L'espacement entre la fin de l'image et la suite (bouton alternative textuelle ou contenu suivant) est trop grand dans le cas où on a pas de caption

🥓 Proposition

Ne pas afficher la figcaption si on en a pas.

🧃 Remarques

RAS

😋 Pour tester

Vérifier que les espacements en dessous des images de bases-clavier-2 et bien-ecrire-son-adresse-mail sont bien cohérents et pas trop grands.

@yannbertrand yannbertrand self-assigned this Mar 7, 2025
@yannbertrand yannbertrand requested a review from a team as a code owner March 7, 2025 10:55
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@clemlatz clemlatz force-pushed the fix-module-element-image-ficaption-spacing branch from 131366c to f121c26 Compare March 7, 2025 14:12
Copy link
Member

@clemlatz clemlatz left a comment

Choose a reason for hiding this comment

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

Je ne suis pas sûr de comment "vérifier que les espacements en dessous des images de sont bien cohérents et pas trop grands." mais ça m'a l'air ok 🤷

PS : j'ai ajouté des ? dans la condition de hasCaption pour réparer les tests, mais peut-être que les propriétés legend et licence sont censées toujours être là, auquel cas il faudrait plutôt corriger le test.

@yannbertrand yannbertrand force-pushed the fix-module-element-image-ficaption-spacing branch from f121c26 to fe8f12c Compare March 7, 2025 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants