Skip to content

[Console] fix buildTableRows when Colspan is use with content too long #30852

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

Merged
merged 1 commit into from
Apr 8, 2019
Merged

[Console] fix buildTableRows when Colspan is use with content too long #30852

merged 1 commit into from
Apr 8, 2019

Conversation

Raulnet
Copy link
Contributor

@Raulnet Raulnet commented Apr 3, 2019

Q A
Branch? 4.2 for bug fixes
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes ( new test added TableTest::testWithColspanAndMaxWith)
Fixed tickets #30701
License MIT
Doc PR no

@nicolas-grekas nicolas-grekas added this to the 4.2 milestone Apr 3, 2019
@nicolas-grekas nicolas-grekas changed the title fix buildTableRows when Colspan is use with content too long [Console] fix buildTableRows when Colspan is use with content too long Apr 4, 2019
@Raulnet
Copy link
Contributor Author

Raulnet commented Apr 5, 2019

Status: needs review

Copy link
Member

@fabpot fabpot left a comment

Choose a reason for hiding this comment

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

It looks like 3.4 has the same issue, in which case this PR should be rebased on 3.4.

@Raulnet
Copy link
Contributor Author

Raulnet commented Apr 8, 2019

Status: needs review

@fabpot
Copy link
Member

fabpot commented Apr 8, 2019

Thank you @Raulnet.

@fabpot fabpot merged commit 1cf9659 into symfony:4.2 Apr 8, 2019
fabpot added a commit that referenced this pull request Apr 8, 2019
…ent too long (Raulnet)

This PR was merged into the 4.2 branch.

Discussion
----------

[Console] fix buildTableRows when Colspan is use with content too long

| Q             | A
| ------------- | ---
| Branch?       |  4.2 for bug fixes
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes    ( new test added  TableTest::testWithColspanAndMaxWith)
| Fixed tickets |  #30701
| License       | MIT
| Doc PR        | no

<!-- fix for keeping ColumnMaxwith when Content is too long

Commits
-------

1cf9659 fix buildTableRows when Colspan is use with content too long
@Raulnet Raulnet deleted the fix/30701/console_table_colspan_max_with branch April 15, 2019 12:43
@fabpot fabpot mentioned this pull request Apr 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants