Skip to content

Exposing zoomFactor in options#162

Merged
marcbachmann merged 2 commits intomarcbachmann:masterfrom
hackjutsu:exposing_zoomFactor
Aug 17, 2016
Merged

Exposing zoomFactor in options#162
marcbachmann merged 2 commits intomarcbachmann:masterfrom
hackjutsu:exposing_zoomFactor

Conversation

@hackjutsu
Copy link
Copy Markdown
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
Copy Markdown
Owner

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

@hackjutsu
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Owner

Released in v2.1.0

@hackjutsu
Copy link
Copy Markdown
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