-
Notifications
You must be signed in to change notification settings - Fork 96
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
RuntimeException when getting the height of a complicated table #110
Comments
I see that the problem is the filter, and row 2 doesn't have a cell with a row span of 1. But I think it's a valid table though? |
Wow. With implementing row spanning I really opened the Pandora's box... 😄 Interesting indeed. I will have a look. But no idea if I am able and willing to fix this 😉 |
Regarding your picture of the table where each row is colored: There is indeed a difficulty with calculating the height of the yellow part in line 2: What height should it get? The base height of the regular cells of row 2? But what is it if there is not at least one regular cell in there (i.e. a cell with row span One option would be to set it to some (configurable?) default height like so:
I am not quite sure yet though if there is any bigger drawback then but can it be worse than a |
Fixed in v0.8.2 using the approach described above. |
Hi
Please don't hate me 😋
The idea of making a general splitter is riping in my head, but it needs some time. In the meanwhile, I found another bug.
I'm trying to split my own tables over different pages with A LOT of extra code to do that, but now I found a table that crashes when you ask for it's height. Without height, I can't split.
Maybe interesting, the height is on purpose bigger than one page, that is why Lorem Ipsum is fed in 😄
The code crashes here, in Row.getHeight()

Structure of the table looks like this, each color is a row.

In words, it looks like this
or what you would like more, in code:
The text was updated successfully, but these errors were encountered: