When using pdf.js, there is no restriction on width and height.
This makes that all pages are shown, taking a lot of vertical space.

Patch adds restrictions, but if height is left empty, the current behavior of showing all pages will be kept, in case someone wants that.

Original report:

In our case, with PDF.js, all pages of a PDF are shown, equal what height we setup in the display-settings. We would expect, that the height restrict the viewer to show all pages of the PDF at once?

CommentFileSizeAuthor
#2 3074904-restrict-pdfjs-dimensions-2.patch1.88 KBpenyaskito

Comments

Rainer F. Gottlieb created an issue. See original summary.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito as a volunteer commented
Status: Active » Needs review
FileSize
3074904-restrict-pdfjs-dimensions-2.patch1.88 KB

Attached patch.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito as a volunteer commented
Title: All pages of PDF shown (pdf.js) » Restrict width and height if specified when using pdf.js so not all pages are shown
Issue summary: View changes

  • penyaskito committed b686d3e1 on 8.x-1.x 
    Issue #3074904 by penyaskito, Rainer F. Gottlieb: Restrict width and...
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito as a volunteer commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.