Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

save_html() now works correctly with components and pages with non-default theme values #823

Merged
merged 8 commits into from
Oct 11, 2023

Conversation

cpsievert
Copy link
Collaborator

@cpsievert cpsievert commented Oct 5, 2023

Closes #477. Requires rstudio/htmltools#411

@@ -32,7 +32,7 @@ Imports:
base64enc,
cachem,
grDevices,
htmltools (>= 0.5.4.9000),
htmltools (>= 0.5.6.9001),
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

main should already be on 0.5.6.9000 🙈

@cpsievert cpsievert requested a review from gadenbuie October 6, 2023 21:03
@cpsievert cpsievert marked this pull request as ready for review October 11, 2023 14:36
Copy link
Member

@gadenbuie gadenbuie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@cpsievert cpsievert merged commit 0d08ff5 into main Oct 11, 2023
@cpsievert cpsievert deleted the save-html branch October 11, 2023 14:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

card() can't be saved directly to HTML with save_html()
2 participants