Skip to content

Don’t include HTML dependencies (from R execution) unless we’re targe…#1381

Merged
jjallaire merged 2 commits intomainfrom
bugfix/htmldep
Jul 13, 2022
Merged

Don’t include HTML dependencies (from R execution) unless we’re targe…#1381
jjallaire merged 2 commits intomainfrom
bugfix/htmldep

Conversation

@dragonstyle
Copy link
Collaborator

…ting HTML output

This should fix #1378

Testing with:

---
always_allow_html: true
---

```{r}
htmltools::tagList(rmarkdown::html_dependency_font_awesome())
htmltools::div(class="dummy", 
               htmltools::p("HTML content"))
```


```{=html}
<i class="fas fa-user"></i>
```

Which should properly output a font awesome icon for HTML while not including any HTML output (including in header) for pdf/latex.

@dragonstyle dragonstyle requested review from cderv and jjallaire July 13, 2022 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HTML dependencies are not filtered out from being included in PDF output

2 participants