Skip to content

Conversation

@cderv
Copy link
Collaborator

@cderv cderv commented Jul 25, 2025

No description provided.

@github-actions github-actions bot temporarily deployed to pull request July 25, 2025 14:35 Inactive
@github-actions
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1727.quarto.org

🔄 Modified Documents

@github-actions github-actions bot temporarily deployed to pull request July 25, 2025 15:06 Inactive
@github-actions
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1727.quarto.org

🔄 Modified Documents

@github-actions github-actions bot temporarily deployed to pull request July 25, 2025 16:52 Inactive
@github-actions
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1727.quarto.org

🔄 Modified Documents

@github-actions github-actions bot temporarily deployed to pull request July 25, 2025 18:12 Inactive
@github-actions
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1727.quarto.org

🔄 Modified Documents

Copy link
Collaborator

@cwickham cwickham left a comment

Choose a reason for hiding this comment

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

Caught a couple of things on a read through. Looks great!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you think people associate the hex clearing with the R package (as opposed to Quarto in general)? I wonder if we should add "R package" somewhere.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I had assume that hex logo was a good reference to package, but I also did not think outside of the R community and this blog post will be published to Quarto blog. 🤔

I see how I can modify. If you have idea for other thumbnail look, please do share. It is not my strong suits and I tried something very simple 😅

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I changed the image

Thanks @cwickham!

Co-authored-by: Charlotte Wickham <charlotte.wickham@posit.co>
@github-actions github-actions bot temporarily deployed to pull request July 28, 2025 10:02 Inactive
@github-actions
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1727.quarto.org

🔄 Modified Documents

@gordonwoodhull
Copy link
Contributor

gordonwoodhull commented Jul 28, 2025

Added

Apply Light and Dark Themes to Plots and Tables

@github-actions github-actions bot temporarily deployed to pull request July 28, 2025 14:53 Inactive
@github-actions
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1727.quarto.org

🔄 Modified Documents

@cderv
Copy link
Collaborator Author

cderv commented Jul 28, 2025

Thank you gordon !

@github-actions github-actions bot temporarily deployed to pull request July 28, 2025 16:11 Inactive
@github-actions
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1727.quarto.org

🔄 Modified Documents

@github-actions github-actions bot temporarily deployed to pull request July 28, 2025 16:56 Inactive
@github-actions
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1727.quarto.org

🔄 Modified Documents

@github-actions github-actions bot temporarily deployed to pull request July 28, 2025 18:48 Inactive
@github-actions
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1727.quarto.org

🔄 Modified Documents

@cderv cderv marked this pull request as ready for review July 28, 2025 18:58
@cderv cderv merged commit 1ac63a3 into main Jul 28, 2025
4 checks passed
@cderv cderv deleted the blog/r-package-1.5 branch July 28, 2025 18:58
@github-actions
Copy link
Contributor

Successfully created backport PR for prerelease:

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.

4 participants