-
Notifications
You must be signed in to change notification settings - Fork 188
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
Clarify revealjs supported plugins list and attributes #464
Comments
Version 4.1.0 has built-in support for zoom, notes and marked/markdown plugins: asciidoctor-reveal.js/lib/asciidoctor-revealjs/converter.rb Lines 201 to 213 in f8f549c
reveal.js plugins system changed quite a bit in reveal.js 4: https://github.com/hakimel/reveal.js/tree/4.3.0/plugin
In Asciidoctor reveal.js 5.0.0, we support the notes, search and zoom plugins. We also decided to drop the markdown plugin since the content is already written in AsciiDoc (see #432). Feel free to improve the 4.1 documentation by using the "Edit this Page" button on the top right corner. We could also improve the documentation for 5.0.0: https://github.com/asciidoctor/asciidoctor-reveal.js/blob/master/docs/modules/converter/pages/revealjs-plugins.adoc by mentioning which plugins are supported and why markdown, highlight and math plugins are superfluous. |
Thank you for your quick and detailed response. I'll process this and will try to help on documenting this ! |
Thank you again for this wonderful tool 🤗
I have a hard time figuring how to enable plugins.
First, the documentation speak about attributes that should be
enable/disable
and the example saysdisable[d]
.Then we know that
zoom
andnotes
are there by default, and indeed, I see inclusion when debugging in browser :Then I download revealjs, checkout the appropriate tag (3.9.2) maching my asciidoctor-revealjs version, list every plugin in the folder, and wild guess some attributes to test :
But then only one more (markdown) seems to be activated :
Could the documentation be more precise on this point, please ? 🙏
The text was updated successfully, but these errors were encountered: