Skip to content
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

[Feature] go-table wrapContent #820

Merged
merged 1 commit into from
Aug 20, 2021
Merged

Conversation

grahamhency
Copy link
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our guidelines
    - [ ] Tests for the changes have been added
  • Docs have been added or updated

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Style Update (CSS)
  • Other... Please describe:

What is the current behavior?

Resolves #811

What is the new behavior?

Added a binding to the go-table component called wrapContent that controls whether the content in all of the table cells wraps or not.

Added a binding to the go-table-column component called wrapContent that controls the content wrapping for that individual column. Note: this binding will override the wrapContent binding on the go-table for that individual column.

Does this PR introduce a breaking change?

  • Yes
  • No

@grahamhency grahamhency requested a review from swimtech20 August 12, 2021 18:09
@grahamhency grahamhency added the feature New feature request label Aug 12, 2021
@grahamhency grahamhency requested review from schneiderduketangoe and removed request for swimtech20 August 17, 2021 16:59
Copy link
Contributor

@schneiderduketangoe schneiderduketangoe 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 Graham. Small nit-picky comment but I'm not gonna let that hold back the approval

@mobi mobi deleted a comment from schneiderduketangoe Aug 18, 2021
@grahamhency
Copy link
Contributor Author

@schneiderduketangoe I had to rebase and also explained my thought process, would you mind approving again?

Copy link
Contributor

@schneiderduketangoe schneiderduketangoe left a comment

Choose a reason for hiding this comment

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

Good to go!

@grahamhency grahamhency merged commit e4afdcd into dev Aug 20, 2021
@grahamhency grahamhency deleted the feature_table_wrapContent branch August 20, 2021 14:48
@grahamhency grahamhency mentioned this pull request Aug 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Go Table Column Wrap
2 participants