Skip to content

fix(pdf): clean HTML tables to fit PDF page width to avoid overflow#51

Merged
Rom1-B merged 3 commits intopluginsGLPI:10.0/bugfixesfrom
MyvTsv:ticket39537_3
Nov 12, 2025
Merged

fix(pdf): clean HTML tables to fit PDF page width to avoid overflow#51
Rom1-B merged 3 commits intopluginsGLPI:10.0/bugfixesfrom
MyvTsv:ticket39537_3

Conversation

@MyvTsv
Copy link

@MyvTsv MyvTsv commented Nov 12, 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 !39537
  • This fix removes style attributes that affect cell width and height to prevent the table from extending beyond the PDF file.

Complete: #50

Screenshots (if appropriate):

Original Table:
image

PDF Rendering Before fix:
image

PDF Rendering After Fix:
image

@MyvTsv MyvTsv requested review from Rom1-B and stonebuzz November 12, 2025 09:13
@MyvTsv MyvTsv self-assigned this Nov 12, 2025
@MyvTsv MyvTsv marked this pull request as ready for review November 12, 2025 09:15
Copy link

@stonebuzz stonebuzz left a comment

Choose a reason for hiding this comment

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

LGTM

@Rom1-B Rom1-B merged commit 698cd94 into pluginsGLPI:10.0/bugfixes Nov 12, 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