Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

font subsetting in PDF/A PDF/X #1315

Open
danielrona opened this issue Nov 12, 2020 · 3 comments · May be fixed by #1940
Open

font subsetting in PDF/A PDF/X #1315

danielrona opened this issue Nov 12, 2020 · 3 comments · May be fixed by #1940
Labels
feature/enhancement fonts

Comments

@danielrona
Copy link

Is there a specifc reason for this?

mpdf/src/Writer/FontWriter.php

Lines 65 to 67 in 822c72c

if ($this->mpdf->PDFA || $this->mpdf->PDFX) {
$asSubset = false;
}

If not I would like to submit a pull request that introduces a config variable for having Font subsets for PDF/A or PDF/X files

PDF/A - Page 6

PDF/A also places higher demands on
the information it contains. All required
fonts (or at least all glyphs for the specific
characters used) must be embedded within
the PDF.

PDF/X - Page 6

Fonts must be embedded (either fully
embedded, or as an embedded subset
in which all characters used in the text
are present).
@finwe
Copy link
Member

finwe commented Nov 13, 2020

Don't know. Thanks for the links, feel free to prepare the PR.

@NicolasCARPi
Copy link

NicolasCARPi commented Oct 24, 2023 •
edited

Hello,

This feature would be interesting for us. Would simply deleting this condition be enough? I don't think there is a need for another setting.

@finwe can you ask your past self from 5 years ago what he had in mind writing this if ? ;)

@finwe
Copy link
Member

finwe commented Oct 24, 2023

I still don't know. I am almost completely sure that I didn't write this if at all.

@danielrona danielrona linked a pull request Oct 25, 2023 that will close this issue
Open
danielrona added a commit to danielrona/mpdf that referenced this issue Nov 16, 2023
@danielrona
fix: FontWriter PDF/A/X Font Subsetting should be enabled (Closes mpd…
99f4751 
…f#1315)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature/enhancement fonts
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: FontWriter PDF/A/X Font Subsetting should be enabled (Closes #1315) danielrona/mpdf
3 participants
@finwe @danielrona @NicolasCARPi