astefanutti / decktape

PDF exporter for HTML presentations
MIT License
2.19k stars 176 forks source link

Reveal remove transition #258

Closed nicojs closed 1 year ago

nicojs commented 1 year ago

I think we should remove the transition when navigating reveal slides:

    return this.page.evaluate(fragments => Reveal.configure({
        controls  : false,
        progress  : false,
        fragments : fragments,
+       transition: 'none',
      }),

Disabling the transition will disable the next-slide animation. This way, users can configure a shorter "pause", resulting in a much faster export.

Code here:

https://github.com/astefanutti/decktape/blob/2474dddfbbedf1eed93320423221ff99522e84cb/plugins/reveal.js#L33-L37

astefanutti commented 1 year ago

I agree it makes sense.