Skip to content

Ajuste le tableau récapitulatif BTP : hauteurs adaptatives et colonnes réduites#29

Merged
mapiolca merged 2 commits intomainfrom
2026-02-02-modifier-tableau-summary-et-colonnes
Feb 2, 2026
Merged

Ajuste le tableau récapitulatif BTP : hauteurs adaptatives et colonnes réduites#29
mapiolca merged 2 commits intomainfrom
2026-02-02-modifier-tableau-summary-et-colonnes

Conversation

@mapiolca
Copy link
Copy Markdown
Owner

@mapiolca mapiolca commented Feb 2, 2026

Motivation

  • Rendre la colonne "Description" plus large en réduisant légèrement la largeur des colonnes de date/montants et permettre que les lignes du tableau récapitulatif s'ajustent à leur contenu multi‑ligne pour éviter les coupures de texte.

Description

  • Réduit la constante de largeur des colonnes fixes en résumé BTP de 25 à 23 pour agrandir la colonne description dans _tableauBtpOrdersSummary du fichier core/modules/facture/doc/pdf_crabe_btp_inpose.modules.php.
  • Calcule désormais une hauteur par ligne ($row_heights) en utilisant getStringHeight quand disponible afin d'adapter la hauteur des lignes au contenu (description et date) et additionne ces hauteurs pour obtenir la hauteur totale du tableau.
  • Met à jour le dessin de la grille pour tracer les lignes horizontales en fonction des hauteurs calculées au lieu d'utiliser une hauteur fixe par ligne, et adapte les appels MultiCell pour utiliser la hauteur de cellule correcte lors du rendu des colonnes date/HT/TVA/TTC.
  • Retourne la hauteur réelle calculée ($height) pour préserver le positionnement correct des éléments suivants dans le PDF.

Testing

  • Aucun test automatique n'a été exécuté pour cette modification (aucune exécution demandée).

Codex Task

@mapiolca mapiolca merged commit ae21800 into main Feb 2, 2026
1 check passed
@mapiolca mapiolca deleted the 2026-02-02-modifier-tableau-summary-et-colonnes branch February 2, 2026 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant