Skip to content

Fixed colspan issues with multiple render() calls #14983

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

Closed
wants to merge 3 commits into from
Closed

Fixed colspan issues with multiple render() calls #14983

wants to merge 3 commits into from

Conversation

jaytaph
Copy link
Contributor

@jaytaph jaytaph commented Jun 14, 2015

Q A
Bug fix? [yes]
New feature? [no]
BC breaks? [no]
Deprecations? [no]
Tests pass? [yes]
Fixed tickets #14982
License MIT
Doc PR

@aitboudad
Copy link
Contributor

@jaytaph I made an alternative way to solve the issue (see #14991) without using originalRows.
Can I close this PR in favor of #14991 ?

@jaytaph
Copy link
Contributor Author

jaytaph commented Jun 15, 2015

@aitboudad Of course :) I was thinking of a similar solution, as I did not really see a reason why the newly build rows should be stored back in the original property, but I did not want to change that as there might have been a reason I did not see or understand.

But if there isn't any, this way might be cleaner.

@jaytaph jaytaph closed this Jun 15, 2015
fabpot added a commit that referenced this pull request Jun 22, 2015
This PR was merged into the 2.8 branch.

Discussion
----------

[Console][Table] allow multiple render() calls.

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Fixed tickets  | ~
| Tests pass?   | yes
| License       | MIT

This is a cherry-pick of #14983.

Commits
-------

b93bcc1 Fixed colspan issues with multiple render() calls
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants