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.

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

Fixing output in Safari #36

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fixing output in Safari #36

wants to merge 2 commits into from

Conversation

leek
Copy link

@leek leek commented Aug 21, 2020

The change made in this repository to convert $this->buffer to $this->get_buffer() breaks the form field display in Safari as well as the display in all browsers when using $pdf->Merge(true);.

This update fixes that in addition to a small patch to allow the library to handle absolute paths.

@reddingwebpro
Copy link

reddingwebpro commented Oct 28, 2020

@leek
It looks like it does output in safari, however for me it shows the pdf file before the merge when in Safari, but in Chrome on the PC is shows the properly merged document. Any idea?

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.

2 participants