Add md_extensions option #75
Conversation
|
Forgotten to mention this PR closes #68 |
| from = rmarkdown_format(ifelse(fig_caption, | ||
| "", | ||
| "-implicit_figures")), | ||
| from = rmarkdown_format(paste0( |
There was a problem hiding this comment.
Let's do this the same way that slidy_presentation does (https://github.com/rstudio/rmarkdown/blob/07b238ceee3aa428acdfbb7351c1cfd9ae48e3f7/R/slidy_presentation.R#L95). Basically chain it together with smart then pass the result (along with fig_caption) to from_rmarkdown
There was a problem hiding this comment.
Alright. Do we check Pandoc's version > 2.0 as slidy_presentation does?
For the smart option, revealjs_presentation does not check it, but slidy_presentation does.
Unfortunately, rmarkdown package does not export smart_extiensions nor pandoc2.0.
There was a problem hiding this comment.
I don't think we need to do the v2.0 check.
There was a problem hiding this comment.
Recently, I noticed that the smart option is meaningless.
Maybe it is a choice for revealjs_presentation to drop the smart option for the better connection with Pandoc.
rstudio/rmarkdown#1774
This should have been done this way in #75 when support was added for `md_extensions`
To improve consistency with other formats (e.g.,
rmarkdown::html_document)