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

Extend HTML output for pagebreaks for PDF generation #441

Merged
merged 15 commits into from
Dec 10, 2014

Conversation

chc88
Copy link
Contributor

@chc88 chc88 commented Dec 10, 2014

Make HTML output of pagebreak compatible for the different PDF writers by combining:
http://mpdf1.com/manual/index.php?tid=110 (tested and working)
and
https://code.google.com/p/dompdf/wiki/FAQ (tested and working)
and
http://chandeshparekh.com/css-to-create-new-page-in-php-tcpdf-pdf/

This is a pull-request for this issue: #426

chrissharkman and others added 15 commits June 24, 2014 09:27
it's not perfect, but better than nothing.
Revert "Fix page break sample code block"
Add missing setters for pageSizeW and pageSizeH
Revert "Add missing setters for pageSizeW and pageSizeH"
Make HTML output of pagebreak compatible for the different PDF writers by combining:
http://mpdf1.com/manual/index.php?tid=110 (tested and working)
and
https://code.google.com/p/dompdf/wiki/FAQ (tested and working)
and
http://chandeshparekh.com/css-to-create-new-page-in-php-tcpdf-pdf/
Add new-page function for PDF generation. For multiple PDF-backends - @chc88 PHPOfficeGH-426
Update pdf-generation; check if we're generating a pdf
@Progi1984
Copy link
Member

OK for me :)

Just need my PC for merge the PR.

@Progi1984 Progi1984 merged commit d330e57 into PHPOffice:develop Dec 10, 2014
Progi1984 added a commit that referenced this pull request Dec 10, 2014
Progi1984 added a commit that referenced this pull request Dec 10, 2014
ghost pushed a commit that referenced this pull request Dec 17, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

5 participants