-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: ability to set/override wordWrap and wrapOnWordBoundary options per cell #303
Conversation
Unit tests added. |
I know there is the hyperlink option, but you can imagine instead of urls, there could be a long identifier in the column, like npm package names, etc |
@speedytwenty any chance of this getting pulled in and released? Thanks! |
The CI tests didn't execute for this PR. Not sure why or how to induce them. Must be something with first-time contributors and github. Once we can trigger these so we have a record of passing tests, I can get it merged. |
You have to configure your Here's the whole reference for |
FWIW, until you've fixed up your CI issue, I have included the video of my tests passing: test.mp4 |
I apologize for the delay between responses. You've caught me during my peak busy time of the year. I'm quite sure the CI issue is a github bug. I had glanced at the PR before you added tests, and there was an option to approve you as a new contributor to execute the GA workflow but that option disappeared after you pushed another commit. I suggest we try this:
|
@speedytwenty thanks! I've squashed all 3 commits into one. I see "1 workflow awaiting approval" so I think it may have worked |
@speedytwenty thank you for the merge! any chance there can be a new release? |
Description
This adds the ability to set
wordWrap
andwrapOnWordBoundary
options per cell, instead of it being set globally via the table options.Use case is for when you want one column to have
wrapOnWordBoundary=false
, and another column withwrapOnWordBoundary=true
(see screenshot and sample code)Sample Code
Screenshot