Skip to content

fix(pdf): adjust table cell width#50

Merged
stonebuzz merged 3 commits intopluginsGLPI:10.0/bugfixesfrom
MyvTsv:ticket39537_2
Nov 3, 2025
Merged

fix(pdf): adjust table cell width#50
stonebuzz merged 3 commits intopluginsGLPI:10.0/bugfixesfrom
MyvTsv:ticket39537_2

Conversation

@MyvTsv
Copy link

@MyvTsv MyvTsv commented Nov 3, 2025

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes !39257)
  • When a table is copied/pasted, some cells inherit a width: 0px style.
    This causes problems because the TCPDF library ignores these cells when generating the PDF.

This fix detects cells whose CSS style contains width: 0px and removes this property before PDF printing.
This ensures that all cells are displayed correctly in the final document rendering.

Screenshots (if appropriate):

Original:
image

PDF Before fix:
image

PDF After fix:
image

@MyvTsv MyvTsv self-assigned this Nov 3, 2025
@MyvTsv MyvTsv requested review from Rom1-B and stonebuzz November 3, 2025 09:25
@MyvTsv MyvTsv marked this pull request as ready for review November 3, 2025 09:25
Co-authored-by: Romain B. <8530352+Rom1-B@users.noreply.github.com>
@stonebuzz stonebuzz merged commit 8a8dfa6 into pluginsGLPI:10.0/bugfixes Nov 3, 2025
3 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