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

Added cmd-line option for stripping fragments and fixing relative paths in markdown/img images #14

Merged
merged 4 commits into from
Oct 7, 2017

Conversation

liuq
Copy link

@liuq liuq commented Oct 6, 2017

Added a command-line option (-h --handouts) for stripping pauses from presentations (which are useful for getting handouts).

Moreover, there's a fix for relative image paths expressed either as markdown images (i.e., with the syntax) or as html images (i.e., with the <img> tag).
In particular, the relative image paths in case of export and pdf are resolved to the absolute paths in order to properly have the image included in the output.

Luca Di Gaspero added 3 commits October 6, 2017 14:25
…which are useful for getting handouts).

Moreover, there's a fix for relative image paths expressed either as markdown images (i.e., with the ![]() syntac) or as html images (i.e., with the <img> tag).
In particular, the relative image paths in case of export and pdf are resolved to the absolute path in order to properly have the image included in the output.
@sinedied
Copy link
Owner

sinedied commented Oct 7, 2017

Thanks for the PR, those are nice additions!

Before merging, could you change 2 little things?

  • Change the doc a bit for the handouts option: the correct term for what your are calling pauses is fragments 😉 (I had to look at the code to understand what you were referring to)
  • Remove the version bump in package.json, I prefer to have a separate bump commit for traceability. Besides, this would be a minor version bump following semver conventions.

…ses.

Removed the version bump in package.json and CHANGELOG.md
@liuq
Copy link
Author

liuq commented Oct 7, 2017

I did it. By the way, if you're interested, I could also work on a further addition to inline external images referred in the markdown file (currently they are just externally linked, that's the reason why I was forced to fix the path). Would you be interested?

@liuq liuq changed the title Added cmd-line option for stripping pauses and fixing relative paths in markdown/img images Added cmd-line option for stripping fragments and fixing relative paths in markdown/img images Oct 7, 2017
@sinedied
Copy link
Owner

sinedied commented Oct 7, 2017

Thanks, it's perfect 👍

Inlining external images would be a great addition, I would gladly accept the PR ❤️

@sinedied sinedied merged commit 4ea50a8 into sinedied:master Oct 7, 2017
@sinedied
Copy link
Owner

sinedied commented Oct 7, 2017

Just for info, I changed the usage doc a bit to fit 80 chars wide terminals and fixed the handouts options being enabled by default for pdf export (it could not be disabled).

I published a new 2.2.0 version with your changes.

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