-
-
Notifications
You must be signed in to change notification settings - Fork 121
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
Add ability to include headers and footers in exported HTML / Markdown #66
base: master
Are you sure you want to change the base?
Conversation
fd8ab03
to
68657ab
Compare
I seem to have some Travis issues, but I don't think I've changed anything that would materially affect the build process, and it seems to be working on 2.6! Any ideas? |
Your PR looks great, I can confirm I get the same error on current build. So it has nothing to do with your PR.
I have submitted an issue (#68) |
bbf2588
to
c7f091b
Compare
c7f091b
to
676c721
Compare
@MisterKeefe Travis issues have been fixed. Can you fetch the latest updates? |
@GitBruno thanks for the heads-up, I've pulled them into this branch 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
include headers and footers
Any idea when this will be merged? Also is it functional for node.js? |
This PR introduces two new keyword arguments to the HTML and Markdown conversion API. These arguments allow for the inclusion of headers and footers in the output. By default, the options are False, so existing behaviour is not affected.
include_headers_and_footers
: If enabled, includes header text at start of output, and appends footer text to end.deduplicate_headers_and_footers
: If enabled, headers and footers will be deduplicated based on their text content.I have tried to keep the PR in the spirit of the library, in the sense that it is the information content of headers and footers which is retrieved, rather than any formatting details.
Please let me know what you think!