Expected behaviour re outer table border with nonzero gutter? about fpdf2 HOT 5 OPEN

My preference for this season would be:

:-)

But for the rest of the year I think your proposal is a good one.
Just one remark/question: Is the padding supposed to be to the centerline of the border or to the edge of the border? In the picture you posted it appears to be the centerline:

from fpdf2.

When the outer border width is not specified then I think the border should be fragmented.

I don't think there was really a use case for the combination of outer border width and padding when I implemented it. Having it continuous would be more efficient.

from fpdf2.

Hi @mjasperse and thank you for this report! 👍

To me this seems like a bug.
I looked at the existing test reference PDF files, and to me it would make more sense for them to have an "unbroken" outer border.

Pinging @RubendeBruin about this, as he contributed most of the padding implementation, and may have extra useful information to share regarding this 😊

But it'd be nice to see how you fixed that, so feel free to submit a PR!

from fpdf2.

Before I submit a PR, I want to check what you think of the following change to handle this combination of options:

This adds the gutter space inside the outer border, which is compatible with how HTML renders tables with nonzero border-spacing.

from fpdf2.

@RubendeBruin thanks for the observation! I added a padding amount of outer_border_width/2 so that the spacing is more consistent as you pointed out.

from fpdf2.