Skip to content

Line br word#1983

Merged
rich-iannone merged 9 commits intorstudio:masterfrom
thebioengineer:line_br_word
Apr 23, 2025
Merged

Line br word#1983
rich-iannone merged 9 commits intorstudio:masterfrom
thebioengineer:line_br_word

Conversation

@thebioengineer
Copy link
Copy Markdown
Collaborator

Summary

Thank you for contributing to gt! To make this process easier for everyone, please explain the context and purpose of your contribution. Also, list the changes made to the existing code or documentation.

This PR resolves 2 issues listed for Word. First is resolve the issue where all outputs for word include "Table #" prepending the caption. This is now optional (default is still TRUE).

Text processing code was updated to incorporate the proposed "universal linebreak" using the
tag for md and html formatting.

Related GitHub Issues and PRs

Checklist

@thebioengineer
Copy link
Copy Markdown
Collaborator Author

@ddsjoberg

Copy link
Copy Markdown
Collaborator

@olivroy olivroy left a comment

Choose a reason for hiding this comment

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

Looks good to me ! thanks very much for taking this on. Minor style comments only.

Co-authored-by: olivroy <52606734+olivroy@users.noreply.github.com>
@thebioengineer
Copy link
Copy Markdown
Collaborator Author

@olivroy - incorporated the suggestions. ready once the actions complete I think to merge in.

Copy link
Copy Markdown
Member

@rich-iannone rich-iannone left a comment

Choose a reason for hiding this comment

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

LGTM!

@rich-iannone
Copy link
Copy Markdown
Member

@thebioengineer Apologies for taking so long on this (I was OOO for a bit when this came in). Merging now!

@rich-iannone rich-iannone merged commit 5db2f92 into rstudio:master Apr 23, 2025
12 checks passed
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.

3 participants