Skip to content

PDF: wrap long CPE strings, fix page-edge truncation (#167) - #236

Merged
techmore merged 1 commit into
mainfrom
fix/pdf-table-truncation
Aug 22, 2026
Merged

PDF: wrap long CPE strings, fix page-edge truncation (#167)#236
techmore merged 1 commit into
mainfrom
fix/pdf-table-truncation

Conversation

@techmore

Copy link
Copy Markdown
Owner

Last cosmetic item from the #167 v2 audit: long CPE strings were cut off at the right page edge.

Change

Print-media CSS in nmap-pdf-olive-legacy.xsl:

  • Constrain max-w-7xl content to full width for print
  • Smaller table typography (9px) with tighter padding
  • word-break: break-all on monospace cells so CPE identifiers wrap within their columns

Verification

Visually inspected regenerated PDF (5 pages): cpe:/a:openbsd:openssh:10.3 fully visible, wrapped within its column, no edge truncation. Layout and readability otherwise unchanged.

Refs #167

…ion (#167)

Adds a print-media CSS block: constrains content width, shrinks table typography,
and enables word-break on monospace cells so long CPE identifiers wrap within their
columns. Verified visually - cpe:/a:openbsd:openssh:10.3 now fully visible on one
page with no edge cut-off.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@techmore
techmore merged commit 0a73e94 into main Aug 22, 2026
2 of 3 checks passed
@techmore
techmore deleted the fix/pdf-table-truncation branch August 22, 2026 23:56
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.

1 participant