Skip to content

fix(virtues): compress image section to prevent text overflow (#252) - #285

Merged
jsboige merged 1 commit into
masterfrom
fix/252-virtues-text-overflow
May 16, 2026
Merged

fix(virtues): compress image section to prevent text overflow (#252)#285
jsboige merged 1 commit into
masterfrom
fix/252-virtues-text-overflow

Conversation

@jsboige

@jsboige jsboige commented May 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add max-height: 50% to .imageSection in Virtues card template CSS to limit image area height
  • Add flex-grow: 1 to .texte to let description text fill the freed space
  • Fixes text overflow/clipping on Virtues cards with long descriptions (e.g. EN "Intellectual honesty", EN "Fruitful exchange", RU "Уместный аргумент")

Test plan

  • Run pipeline to regenerate Virtues card images
  • Visual validation: check that description text is no longer clipped on cards with long descriptions
  • Visual validation: check that image still renders correctly (not squished)
  • Verify all 8 Virtues families render correctly with family colors

Closes #252

🤖 Generated with Claude Code

Add max-height: 50% to .imageSection and flex-grow: 1 to .texte in
Virtues card template. This limits the image area to half the card
height, giving more space to description text that was being clipped.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

@clusterManager-Myia clusterManager-Myia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Review automatique Hermes 🔍

Verdict : OK (comment only — self-review)

Analyse du diff sur Argumentum_Virtues_Face_fr.json :

  • Le fichier est entièrement régénéré (1 ligne minifiée → 1 ligne minifiée)
  • Comparaison côté à côté du CSS et du CSV : aucune différence fonctionnelle détectée
  • Le contenu des styles, la structure des cartes, le CSV et le template Mustache sont identiques
  • Il s'agit probablement d'un reformatage ou d'une régénération automatique (outil de build, lint JSON, etc.)

Observations mineures :

  • Le fichier reste sans newline final (No newline at end of file) — c'est pré-existant et non introduit par cette PR
  • Le numéro de pk 19 (présentationIntègre > Humour > Autodérision) manque toujours dans le CSV (pk skip 18→20) — pré-existant

Pas de risque de régression. Bon pour merge.

— Hermes (myia-po-2026) [CRON:review-pr 16:35Z]

@jsboige

jsboige commented May 16, 2026

Copy link
Copy Markdown
Contributor Author

LGTM ✅ — fix conforme à la recommandation cycle 12.

Vérification du diff CSS (extrait sous la couche minifiée)

.imageSection {
    min-height: 0;
    flex-shrink: 1;
+   max-height: 50%;
    display: flex;
    ...
}

.texte {
+   flex-grow: 1;
    flex-shrink: 1;
    ...
}

Note : la review automatique Hermes a manqué ce changement parce que le fichier est minifié sur une seule ligne (diff side-by-side ne décode pas l'intérieur de la string CSS). Vérification manuelle confirme le fix est bien présent et correctement positionné.

Couverture

Reste hors scope (acceptable pour cette PR)

Validation pipeline post-merge

Test plan reste à exécuter (regen + visual check). Si après regen les cartes problématiques sont OK, ferme #252.

Tente merge maintenant.

@jsboige
jsboige merged commit 484970a into master May 16, 2026
1 check passed
@jsboige
jsboige deleted the fix/252-virtues-text-overflow branch June 1, 2026 21:42
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.

fix(virtues): Virtues cards text overflow + missing family colors + compressible image div

2 participants