Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Magazine PDF with side-by-side pages renders off-screen #9359

Closed
dalin- opened this issue Jan 12, 2018 · 6 comments
Closed

Magazine PDF with side-by-side pages renders off-screen #9359

dalin- opened this issue Jan 12, 2018 · 6 comments
Labels

Comments

@dalin-
Copy link

dalin- commented Jan 12, 2018

Attach (recommended) or Link to PDF file here:
https://www.aclu.org/sites/default/files/field_document/aclu_stand_mag_summer_2015.pdf

Configuration:

  • Web browser and its version: FF 58, but it doesn't seem to matter.
  • Operating system and its version: OSX 10.13, but it doesn't seem to matter.
  • PDF.js version: 1.10.88, but that doesn't seem to matter either
  • Is a browser extension: No

Steps to reproduce the problem:

  1. Use the PDF viewer embedded here:
    https://www.aclu.org/other/read-latest-issue-stand
    The cover page will show auto zoom (and I think it is choosing fit-width).
  2. Scroll down to the next page (other than the cover, all pages are side-by-side). Odd pages are rendered off-screen.

image

What is the expected behavior? (add screenshot)

If you play around with opening the PDF viewer sidebar and choosing individual pages and/or switching between Automatic Zoom and Page Width, you can get into a state where the cover page is on its own, but subsequent pages are side-by-side. See screenshots.

image

image

I'm trying to make this the default state when opening the PDF. I've been playing around with the viewer options. The closest that I've gotten is
#page=2&zoom=page-fit,0,1100
which results in this:

image

So close..., but if I add any more top offset it goes back to looking like the original screenshots.

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):
https://www.aclu.org/other/read-latest-issue-stand

I'm not sure if this is a bug, or just some hidden configuration value that I don't know about.

@dalin- dalin- changed the title How to get a magazine PDF with side-by-side pages to display "page-fit"? Magazine PDF with side-by-side pages renders off-screen Jan 12, 2018
@brendandahl
Copy link
Contributor

I haven't looked to deeply yet, but I think it's because we base page-fit off the size of the first page. We could probably do something smarter and look at subsequent page sizes.

@dalin-
Copy link
Author

dalin- commented Jan 16, 2018

So that I can go back to my client with the full range of possible options, does this project have a bug bounty program? Formal or otherwise? The only other offer of a bounty that I could find is
#2594 (comment)
And it's not clear whether or not the bounty was actually claimed.

@brendandahl
Copy link
Contributor

There's no formal bug bounty program. I do believe the one you referenced above was claimed though.

@brendandahl
Copy link
Contributor

Somewhat off topic... I'm not sure what the intentions of the above site, but if it's purely for viewing the PDF online, you (or your client) may want to export that PDF as a PDF optimized for the web. It looks like some of the images in it have a really high DPI and could be much smaller if it only needs to be shown online.

@dalin-
Copy link
Author

dalin- commented Jan 18, 2018

Thanks for your thoughts. The current plan is to modify the PDF, and see how that works.

@Snuffleupagus
Copy link
Collaborator

Duplicate of #5628

@Snuffleupagus Snuffleupagus marked this as a duplicate of #5628 Nov 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants