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

Modularize css-to-pdf #54

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

JoshuaCWebDeveloper
Copy link

@JoshuaCWebDeveloper JoshuaCWebDeveloper commented Nov 28, 2016

I've rewritten css-to-pdf to be able to be imported as a module in various environments.

The largest changes where whitespace/formatting changes. Indentation was currently a mix of tabs, 4-spaces, and 2-spaces, so I replaced all the tabs with 4-spaces to be more consistent. Also, wrapping the existing code in a function made it necessary to indent everything. These two major formatting changes are in their own commits.

I've also created a package.json file which will allow users to install this via npm as Xportability/css-to-pdf. This is also most of the work needed for Issue #31, all that is needed after this is to follow the steps in https://docs.npmjs.com/getting-started/publishing-npm-packages. The version number should be updated when future changes are "released."

This PR should not affect any current users; css-to-pdf can still be included in a <script> tag to create the global xepOnline variable as is currently documented.

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.

1 participant