Merge pull request #12023 from escapewindow/issue9297

Fix auto-rotate for printing landscape PDFs where the first page is also landscape
This commit is contained in:
Tim van der Meij 2020-07-17 19:48:14 +02:00 committed by GitHub
commit d69fb446bf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 5 deletions

View File

@ -167,7 +167,7 @@
},
"enablePrintAutoRotate": {
"title": "Automatically rotate printed pages",
"description": "When enabled, pages whose orientation differ from the first page are rotated when printed.",
"description": "When enabled, landscape pages are rotated when printed.",
"type": "boolean",
"default": false
},

View File

@ -67,8 +67,7 @@ const DEFAULT_CACHE_SIZE = 10;
* @property {boolean} [renderInteractiveForms] - Enables rendering of
* interactive form elements. The default is `false`.
* @property {boolean} [enablePrintAutoRotate] - Enables automatic rotation of
* pages whose orientation differ from the first page upon printing. The
* default is `false`.
* landscape pages upon printing. The default is `false`.
* @property {string} renderer - 'canvas' or 'svg'. The default is 'canvas'.
* @property {boolean} [enableWebGL] - Enables WebGL accelerated rendering for
* some operations. The default value is `false`.
@ -1203,9 +1202,8 @@ class BaseViewer {
if (!this.enablePrintAutoRotate) {
return pagesOverview;
}
const isFirstPagePortrait = isPortraitOrientation(pagesOverview[0]);
return pagesOverview.map(function (size) {
if (isFirstPagePortrait === isPortraitOrientation(size)) {
if (isPortraitOrientation(size)) {
return size;
}
return {