Skip to content

Parbox wrapping text columns#2097

Merged
rich-iannone merged 17 commits intorstudio:masterfrom
thebioengineer:parbox_wrapping_text_columns
Jan 14, 2026
Merged

Parbox wrapping text columns#2097
rich-iannone merged 17 commits intorstudio:masterfrom
thebioengineer:parbox_wrapping_text_columns

Conversation

@thebioengineer
Copy link
Copy Markdown
Collaborator

@thebioengineer thebioengineer commented Jan 5, 2026

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.

The issue is that when a table has really wide text, and if the user either applies a cell alignment or line break the table wont automatically text wrap the rest of the text in late outputs. I can open an issue if needed.

So basically I now resolves it by applying the column width limitations to the multicolumn calls and wrapping the internal text in a parbox. Latex aficionados please correct me if that is the wrong thing to do.

Related GitHub Issues and PRs

  • Ref: #

Checklist

@rich-iannone
Copy link
Copy Markdown
Member

Thanks @thebioengineer , there’s a few test failures. Could you address those before I review the PR?

@thebioengineer
Copy link
Copy Markdown
Collaborator Author

Yes, I am working on them now!

@thebioengineer
Copy link
Copy Markdown
Collaborator Author

hey @rich-iannone - they all pass now. Sorry about that!

@thebioengineer
Copy link
Copy Markdown
Collaborator Author

@rich-iannone -sorry, I need to look into something real quick on my end.

@thebioengineer
Copy link
Copy Markdown
Collaborator Author

@rich-iannone, sorry this took a minute to get prepped again, I added some tests and updated snapshots. basically alignment is complicated, and getting footnotes to be within the shortstacks and parboxes is also not trivial!

but on the bright side, latex also now does left and right aligned footmarks now.

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 rich-iannone merged commit 79b8806 into rstudio:master Jan 14, 2026
12 checks passed
@thebioengineer thebioengineer deleted the parbox_wrapping_text_columns branch January 14, 2026 17:46
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