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

Update to README.md for PDF printing #1823

Merged
merged 2 commits into from
Feb 14, 2017
Merged

Update to README.md for PDF printing #1823

merged 2 commits into from
Feb 14, 2017

Conversation

gilxa1226
Copy link
Contributor

I found that when trying to print to PDF, there was a block of code missing from the HTML, this block is necessary for the PDF layout to be rendered properly and I didn't see it anywhere in the README so I added it in.

@hakimel hakimel merged commit 8b97ff3 into hakimel:master Feb 14, 2017
hakimel added a commit that referenced this pull request Feb 14, 2017
@hakimel
Copy link
Owner

hakimel commented Feb 14, 2017

Thanks! Merged with a couple of tweaks: https://github.com/hakimel/reveal.js/blob/master/README.md#pdf-export

benswift added a commit to benswift/reveal.js that referenced this pull request Apr 4, 2017
* hakimel-gh/master:
  new phantom pdf export script that works with 3.4.0 hakimel#1815
  updated pdf docs
  update readme re: pdf exports hakimel#1823
  Update README.md to include block for pdf printing
  Update README.md
  3.4.1
  adjust pdf margins instead of heights to fix hakimel#1804
  additional fix for empty pages in pdf exports hakimel#1804
  fix empty pages in pdf exports (closes hakimel#1804)
  correct grunt option usage
  For a fragment: allow to show a separate note defined in it
R0bes pushed a commit to R0bes/Terraform-Presentation that referenced this pull request Jun 7, 2021
Update to README.md for PDF printing
R0bes pushed a commit to R0bes/Terraform-Presentation that referenced this pull request Jun 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants