adjust pdf margins instead of heights to fix #1804
This commit is contained in:
		| @@ -90,7 +90,7 @@ ul, ol, div, p { | ||||
| } | ||||
|  | ||||
| .reveal .slides section { | ||||
| 	page-break-after: auto !important; | ||||
| 	page-break-after: always !important; | ||||
|  | ||||
| 	visibility: visible !important; | ||||
| 	display: block !important; | ||||
|   | ||||
| @@ -605,7 +605,7 @@ | ||||
| 			slideHeight = slideSize.height; | ||||
|  | ||||
| 		// Let the browser know what page size we want to print | ||||
| 		injectStyleSheet( '@page{size:'+ pageWidth +'px '+ pageHeight +'px; margin: 0;}' ); | ||||
| 		injectStyleSheet( '@page{size:'+ pageWidth +'px '+ pageHeight +'px; margin: 0 0 -1px 0;}' ); | ||||
|  | ||||
| 		// Limit the size of certain elements to the dimensions of the slide | ||||
| 		injectStyleSheet( '.reveal section>img, .reveal section>video, .reveal section>iframe{max-width: '+ slideWidth +'px; max-height:'+ slideHeight +'px}' ); | ||||
| @@ -652,12 +652,7 @@ | ||||
| 				// so that no page ever flows onto another | ||||
| 				var page = document.createElement( 'div' ); | ||||
| 				page.className = 'pdf-page'; | ||||
|  | ||||
| 				// Reduce total height by 1px so that the page ends before | ||||
| 				// the page, otherwise the page's 'page-break-after' will | ||||
| 				// land on the wrong page | ||||
| 				page.style.height = ( ( pageHeight - 1 ) * numberOfPages ) + 'px'; | ||||
|  | ||||
| 				page.style.height = ( pageHeight * numberOfPages ) + 'px'; | ||||
| 				slide.parentNode.insertBefore( page, slide ); | ||||
| 				page.appendChild( slide ); | ||||
|  | ||||
|   | ||||
		Reference in New Issue
	
	Block a user