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

Extra configurable options #1

Merged
merged 3 commits into from
Apr 9, 2013
Merged

Extra configurable options #1

merged 3 commits into from
Apr 9, 2013

Conversation

joelcox
Copy link
Contributor

@joelcox joelcox commented Apr 3, 2013

Hey Mike, thanks for the great module.

I've added some extra configurable options to the library, especially for PDF output. I haven't fully tested nor documented all the changes yet because I wanted your input first. I made some changes in the way arguments are parsed for greater flexibility. The options object literal now looks like this:

var options = {
    out: './tmp',
    format: 'pdf',
    viewportWidth: 915,
    viewportHeight: 768,
    paperMargin: '2mm',
    paperOrientation: 'portait',
    paperFormat: 'A4',
    phantomBin: __dirname + '/node_modules/phantomjs/bin/phantomjs'
};

@mmoulton
Copy link
Owner

mmoulton commented Apr 6, 2013

Thanks Joel for the pull request. This looks great to me. If you get the tests passing I would be happy to merge it.

@joelcox
Copy link
Contributor Author

joelcox commented Apr 7, 2013

Cool, I'll get this fixed up sometime later.

@joelcox
Copy link
Contributor Author

joelcox commented Apr 9, 2013

I fixed the test and updated the docs. Let me know what you think.

mmoulton added a commit that referenced this pull request Apr 9, 2013
Extra configurable options
@mmoulton mmoulton merged commit 9bd338a into mmoulton:master Apr 9, 2013
@mmoulton
Copy link
Owner

mmoulton commented Apr 9, 2013

Thank you Joel for the great contribution. I will push a new build shortly.

@joelcox
Copy link
Contributor Author

joelcox commented Apr 9, 2013

Awesome!

On Apr 9, 2013, at 7:06 PM, Mike Moulton notifications@github.com wrote:

Thank you Joel for the great contribution. I will push a new build shortly.


Reply to this email directly or view it on GitHub.

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