Fixes issue #1042

[fe-cj]

Fixes #1042

It was not calculating correctly the 'y' at the end of a rowSpan with dontBreakRows.

[liborm85]

PR is in conflict with PR #2775.

[fe-cj]

Ok, will solve the conflicts later.

[fe-cj]

Done

[liborm85]

Cool. Merged. Thanks.

[fe-cj]

It also fixes #2452

[Phi-Hung-1007]

call dontBreakRows: true,
bug :

[fe-cj]

@Phi-Hung-1007 Can you please share a doc definition that we can use on the playground to reproduce the bug? Thanks.

[Phi-Hung-1007]

@fe-cj On the playgound, I use dontBreakRows to avoid breaking rowspan between two pages but still get it, it seems like version 2.0.13 is not applied on playground yet? Thanks

[liborm85]

On playground is latest version 0.2.13.

[Phi-Hung-1007]

@liborm85 So why can't I still use dontBreakRows for rowSpan like in the example above?

[fe-cj]

dontBreakRows applies to single rows, not to rowspanned rows.

[Phi-Hung-1007]

Oh I thought you fixed the dontBreakRows with rowSpan error in the above issue. But when I updated to version 0.2.13 I couldn't even call dontBreakRows with single row because of the error in the first comment.

[fe-cj]

I fixed an error that was adding extra height to the table when combining rowSpan with dontBreakRows.

If you encounter a bug, it would be nice if you open an issue and share a doc definition that others can copy and paste in the playground.

[Phi-Hung-1007]

On the playground, I can't reproduce the error I get in IDE when installing version 0.2.13 and calling dontBreakRows: true
On the Playground:

On my IDE:

[liborm85]

@Phi-Hung-1007 Again... If you found bug, open an issue with example code for reproduce problem runnable on playground or https://jsfiddle.net/ or a similar service, as is described in CONTRIBUTING.md.
Screenshots are not runnable on playground.