make rendering of substeps as separate page optional #174
Conversation
There was a problem hiding this comment.
This would basically covers #24 so it'd be great to have it implemented as well for the Reveal plugin at least.
| }; | ||
|
|
||
| if(options.enableSubsteps===false){ |
There was a problem hiding this comment.
This is specific to Impress so that should better be added to the configure method of the Impress plugin.
The options object can be passed to the Impress plugin constructor.
| @@ -51,6 +51,13 @@ parser.script('decktape').options({ | |||
| default : 0, | |||
| help : 'Duration in milliseconds between the page has loaded and starting to export slides', | |||
| }, | |||
| enableSubsteps: { | |||
There was a problem hiding this comment.
This would basically cover #24. I wonder what could be the best terminology across all the HTML presentation frameworks. Reveal calls it fragments, other refer to incremental display of slide content...
| @@ -22,9 +22,15 @@ class Impress { | |||
| }); | |||
| } | |||
|
|
|||
| slideCount() { | |||
| slideCount(enableSubsteps) { | |||
There was a problem hiding this comment.
I think it'd be preferable to have the options object passed to the Impress plugin constructor and remove the extra argument.
| @@ -365,6 +365,7 @@ async function exportSlide(plugin, page, printer, context) { | |||
| printBackground : true, | |||
| pageRanges : '1', | |||
| displayHeaderFooter : false, | |||
| scale: 1.5 | |||
There was a problem hiding this comment.
I think it's related to #107 and specific to Impress. I would suggest we add an option and change the default value for Impress instead of hard-coding the value for all presentation frameworks.
astefanutti
force-pushed
the
master
branch
4 times, most recently
from
a570d12
to
46b23e5
Compare
Sometimes you want to have all the substeps immediately visible in pdf files and not as separate pages for every substep. With these changes, there will be a
--substeps=falseoption to disable generation of pages for each substep.