New behavior of the `knit2html` function - `fragment.only` parameter is not respected anymore

[federicomarini]

Hi,

This issue is related to https://github.com/federicomarini/ideal_serveredition/issues/11

In brief: I have in my shiny app the functionality to render "in the app" a preview of the report.

https://github.com/federicomarini/ideal/blob/a4595a58227d4b2ff8153744285d8385451f5384/R/ideal.R#L4719

This uses the knitr::knit2html function, which had a parameter fragment.only, set to TRUE so that I could simply "embed" the generated html without scrambling up the css of the app.

Now somehow the knit2html does not care about the parameter anymore and it throws the error reported by @tamuanand. If I remove that, I obtain the report preview but in a way that messes up the look of the app itself.

Something like this:

whereas it used to look like this:

Is there a way to keep that functionality? Or alternatively, an even better way to include the "preview on the fly" of the Rmd report?

Thanks in advance!

Federico

---

By filing an issue to this repo, I promise that

• [x] I have fully read the issue guide at https://yihui.org/issue/.
• [x] I have provided the necessary information about my issue.
  • If I'm asking a question, I have already asked it on Stack Overflow or RStudio Community, waited for at least 24 hours, and included a link to my question there.
  • If I'm filing a bug report, I have included a minimal, self-contained, and reproducible example, and have also included xfun::session_info('knitr'). I have upgraded all my packages to their latest versions (e.g., R, RStudio, and R packages), and also tried the development version: remotes::install_github('yihui/knitr').
  • If I have posted the same issue elsewhere, I have also mentioned it in this issue.
• [x] I have learned the Github Markdown syntax, and formatted my issue correctly.

I understand that my issue may be closed if I don't fulfill my promises.

[federicomarini]

If there are more elegant ways of doing this, I am more than happy to learn how it is supposed to be done.

I am not afraid to say my solution was a bit clunky -- yet it worked 😬 Well, until recently at least

[yihui]

Is there a way to keep that functionality? Or alternatively, an even better way to include the "preview on the fly" of the Rmd report?

Yes, please use knit2html(..., template = FALSE) instead of fragment.only = TRUE. Sorry about the trouble!

The change was from the markdown package: https://github.com/rstudio/markdown/releases/tag/v1.3

[federicomarini]

Fantastic, thanks for the clarification - blazing fast AND positive reply, love it.

I did try to track it down into markdown but was not successful in doing that. Testing it right away!

[federicomarini]

... aaaaaand it works like a charm, as expected. Thanks @yihui , I know at least a long time user of ideal who will cheer with me for the quick solution (cc @tamuanand)

[yihui]

Great! Thanks for testing! I also appreciate your patience and understanding! Please do not hesitate to ask when you run into any trouble next time.

[tamuanand]

Hi @federicomarini and @yihui

thanks a lot

[github-actions[bot]]

This old thread has been automatically locked. If you think you have found something related to this, please open a new issue by following the issue guide (https://yihui.org/issue/), and link to this old issue if necessary.