From f76b4fda93a8d8770390d4e89b68f8ac27e32dc9 Mon Sep 17 00:00:00 2001
From: Hakim El Hattab
Date: Fri, 16 Mar 2018 13:52:03 +0100
Subject: remove pdf height offset

---
 js/reveal.js | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

(limited to 'js/reveal.js')

diff --git a/js/reveal.js b/js/reveal.js
index 626ce7a..dcd09c7 100644
--- a/js/reveal.js
+++ b/js/reveal.js
@@ -196,13 +196,6 @@
 			// 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,
 
@@ -668,8 +661,8 @@
 		var slideSize = getComputedSlideSize( window.innerWidth, window.innerHeight );
 
 		// Dimensions of the PDF pages
-		var pageWidth = Math.floor( slideSize.width * ( 1 + config.margin ) ),
-			pageHeight = Math.floor( slideSize.height * ( 1 + config.margin ) );
+		var pageWidth = Math.ceil( slideSize.width * ( 1 + config.margin ) ),
+			pageHeight = Math.ceil( slideSize.height * ( 1 + config.margin ) );
 
 		// Dimensions of slides within the pages
 		var slideWidth = slideSize.width,
@@ -726,7 +719,7 @@
 				// so that no page ever flows onto another
 				var page = document.createElement( 'div' );
 				page.className = 'pdf-page';
-				page.style.height = ( ( pageHeight + config.pdfPageHeightOffset ) * numberOfPages ) + 'px';
+				page.style.height = ( pageHeight * numberOfPages ) + 'px';
 				slide.parentNode.insertBefore( page, slide );
 				page.appendChild( slide );
 
-- 
cgit v1.2.3