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

Strongly pixelated pages in PDF #5354

Closed
deepj opened this issue Sep 27, 2014 · 6 comments · Fixed by #18035
Closed

Strongly pixelated pages in PDF #5354

deepj opened this issue Sep 27, 2014 · 6 comments · Fixed by #18035

Comments

@deepj
Copy link
Contributor

deepj commented Sep 27, 2014

PDF document: http://apple.servispardubice.cz/images/quadro_srpen2014.pdf
Versions: Firefox 33 beta, pdf.js 1.0.837
OS: Mac OS X 10.9.5

Console output:

None

@CodingFabian
Copy link
Contributor

@Snuffleupagus are you sure the image jpge tags are correct? for me it looks more likely to be a font rendering issue due to the unusual page aspect ratio.

@timvandermeij
Copy link
Contributor

Here's another one (page 20 for example): https://www.ideals.illinois.edu/bitstream/handle/2142/25816/ellison-rosu-2011-tr-b.pdf?sequence=4. This seems to be a regression as this wasn't the case before.

@automatedbugreportingfacility

5262e6f is the first bad commit
commit 5262e6f
Author: Yury Delendik
Date: Thu Feb 13 07:01:45 2014 -0600

Set max size for the group

@timvandermeij
Copy link
Contributor

/cc @yurydelendik

@Snuffleupagus Snuffleupagus added this to the 2015 Q1 milestone Jan 23, 2015
@timvandermeij timvandermeij removed this from the 2015 Q1 milestone Apr 29, 2015
@automatedbugreportingfacility

Here's another one (page 20 for example): https://www.ideals.illinois.edu/bitstream/handle/2142/25816/ellison-rosu-2011-tr-b.pdf?sequence=4. This seems to be a regression as this wasn't the case before.

This is actually a different issue, regressed by #4834 -- you may want to open a separate ticket for it. In this bug, drawnHeight is around 9000. Bumping MAX_GROUP_SIZE to 16384 makes it look okay, but I have no idea if it's a desired fix.

@deepj
Copy link
Contributor Author

deepj commented Dec 17, 2015

Hello, any progress on this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants