Re: Patch to support tables with oversize cells
Lennart Regebro via reportlab-users <[email protected]> Thu, 17 Feb 2022 12:55:41 +0100
| Newsgroups | gmane.comp.python.reportlab.user |
|---|---|
| Message-ID | <CAHT-kBcc0WPmDA8wdOHmSqpU_q+eME0kz7n3LmpRCnO1ON=W8A@mail.gmail.com> |
Yeah, definitely looks like somethings up with the spacing there. I should be able to take a look at this this week! Thanks! On Thu, Feb 17, 2022 at 12:34 PM Robin Becker <[email protected]> wrote: > On 01/02/2022 16:34, Lennart Regebro wrote: > > OK, I got a small bit wrong in the logic of when to split in each way, > > fixed now, new patch attached. > > > ........ > > Hi Lennart, I finally got around to making a small example that > illustrates the splitInrow=0 differences. > > You can see the patched (with splitInrow=0) and unpatched results here > > https://www.reportlab.com/ftp/lennart/ > > the generating script is lennart-example.py and the two pdfs illustrate > behaviour unpatched and patched with > splitInRow=0 as an argument. > > I changed the your patch so by default it starts with splitInRow = 0. I > don't think we should change behaviour at all > unless it's done explicitly. > > To my eye it looks as though the patch has changed the behaviour regarding > space around the split table. > -- > Robin Becker >