Skip to content

Conversation

@yihui
Copy link
Collaborator

@yihui yihui commented Mar 28, 2025

Allow users to disable footnotes (this is for the Shiny app; cc @keaven @LittleBeannie).

@yihui yihui marked this pull request as ready for review March 28, 2025 13:26
R/as_gt.R Outdated
footnote <- parts$footnote
for (i in seq_along(footnote$content)) {
att <- footnote$attr[i]
footnotes <- parts$footnotes
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hi @yihui, in gsDesign2, we adhere to the singular form in coding, which aligns with our conventions established a few years ago. If you concur, could you please revert "footnotes" to "footnote" for consistency with the other instances? Thank you!

@yihui yihui force-pushed the disable-gt-footnotes branch from c62fd55 to 71aaba3 Compare March 28, 2025 16:19
@LittleBeannie LittleBeannie self-requested a review March 28, 2025 17:40
Copy link
Collaborator

@LittleBeannie LittleBeannie left a comment

Choose a reason for hiding this comment

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

Thanks, @yihui!

@LittleBeannie LittleBeannie merged commit d0a5897 into Merck:main Mar 28, 2025
7 checks passed
@yihui yihui deleted the disable-gt-footnotes branch March 29, 2025 03:41
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.

2 participants