dirk
/
why-cant-we-deploy-today
1
0
Fork 0
Code Releases Activity

Revert "remove pdf height offset" 

This reverts commit f76b4fda93.
This commit is contained in:
Hakim El Hattab 2018-03-20 10:09:47 +01:00
parent f76b4fda93
commit ea57e697a1
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
js/reveal.js
View File

@ -196,6 +196,13 @@
// to PDF, unlimited by default
pdfMaxPagesPerSlide: Number.POSITIVE_INFINITY,
// Offset used to reduce the height of content within exported PDF pages.
// This exists to account for environment differences based on how you
// print to PDF. CLI printing options, like phantomjs and wkpdf, can end
// on precisely the total height of the document whereas in-browser
// printing has to end one pixel before.
pdfPageHeightOffset: -1,
// Number of slides away from the current that are visible
viewDistance: 3,
@ -661,8 +668,8 @@
var slideSize = getComputedSlideSize( window.innerWidth, window.innerHeight );
// Dimensions of the PDF pages
var pageWidth = Math.ceil( slideSize.width * ( 1 + config.margin ) ),
pageHeight = Math.ceil( slideSize.height * ( 1 + config.margin ) );
var pageWidth = Math.floor( slideSize.width * ( 1 + config.margin ) ),
pageHeight = Math.floor( slideSize.height * ( 1 + config.margin ) );
// Dimensions of slides within the pages
var slideWidth = slideSize.width,
@ -719,7 +726,7 @@
// so that no page ever flows onto another
var page = document.createElement( 'div' );
page.className = 'pdf-page';
page.style.height = ( pageHeight * numberOfPages ) + 'px';
page.style.height = ( ( pageHeight + config.pdfPageHeightOffset ) * numberOfPages ) + 'px';
slide.parentNode.insertBefore( page, slide );
page.appendChild( slide );