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

Exposing zoomFactor in options #162

Merged
merged 2 commits into from
Aug 17, 2016

Conversation

hackjutsu
Copy link
Contributor

zoomFactor is an important factor when rendering webpages, especially when the webpage is exported as png/jpg. Given that the dpi of the images rendered by PhantamJS is relatively low, many clients choose to adjust the zoomFactor to get better image quality.

Therefore, I would like to expose this factor to clients.

Cosmo added 2 commits August 5, 2016 04:21
The zoomFactor is an important factor when rendering the webpage. Please expose it:)
@marcbachmann
Copy link
Owner

Any idea what happened with the dpi setting in phantomjs? Is that working already?
Would be a better solution.

@hackjutsu
Copy link
Contributor Author

Not sure about that. At least for now DPI configuration is not working in its master branch. (Correct me if I'm wrong:))

If we don't configure the zoomFactor, the image's quality is too poor to view, especially on some mobile devices with retina screen.

@marcbachmann
Copy link
Owner

How do you use that? Do you use it only for images or also for pdfs?
IMO it doesn't make any sense for pdfs as it changes the size of them.

@marcbachmann marcbachmann merged commit dbb323d into marcbachmann:master Aug 17, 2016
@marcbachmann
Copy link
Owner

Released in v2.1.0

@hackjutsu
Copy link
Contributor Author

hackjutsu commented Aug 19, 2016

Thanks! We only use it for png/jpeg.

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

Successfully merging this pull request may close these issues.

2 participants