Vertical align

19 views
Skip to first unread message

maartendb

unread,
Jul 12, 2012, 3:52:12 PM7/12/12
to prettyta...@googlegroups.com
Hello,

As an attachment you can find 5 patches (that have to be applied in serial) that will include vertical align support for prettytable.
The patches were created using git-svn. I think they can be applied easily.

A summary of patches:
  1. some lay-out change (tabs to spaces etc.)
  2. allow only one valign per table (not row/column configurable) (+test)
  3. allow configuration of valign per column
  4. allow configuration of valign per row
  5. avoid the print of the table in the test by using a print hook.
Thanks for reading this far

maartendb
0001-fix-tab-removal.patch
0002-feature-vertical-align.patch
0003-fix-valign-configurable-per-column.patch
0004-valign-configurable-per-row.patch
0005-added-hook-to-stdout-in-PrintTest.patch

Luke Maurits

unread,
Jul 15, 2012, 4:17:34 PM7/15/12
to prettyta...@googlegroups.com
Hi martendb,

Thanks very much for contributing these patches. This exact feature (vertical align) was actually next on my list of things to add for the next prettytable (which will hopefully be out soon), so you have very good timing! I will make sure that your work is included in the upcoming 0.7 release and that you are credited appropriately.

Cheers,
Luke


On Thu, 12 Jul 2012 12:52:12 -0700 (PDT)
maartendb <mad...@gmail.com> wrote:

> Hello,
>
> As an attachment you can find 5 patches (that have to be applied in serial)
> that will include vertical align support for prettytable.
> The patches were created using git-svn. I think they can be applied easily.
>
> A summary of patches:
>
> 1. some lay-out change (tabs to spaces etc.)
> 2. allow only one valign per table (not row/column configurable) (+test)
> 3. allow configuration of valign per column
> 4. allow configuration of valign per row
> 5. avoid the print of the table in the test by using a print hook.
>
> Thanks for reading this far
>
> maartendb
>
> --
> You received this message because you are subscribed to the Google Groups "prettytable-devel" group.
> To view this discussion on the web visit https://groups.google.com/d/msg/prettytable-devel/-/RyAMPiRym10J.
> To post to this group, send email to prettyta...@googlegroups.com.
> To unsubscribe from this group, send email to prettytable-de...@googlegroups.com.
> For more options, visit this group at http://groups.google.com/group/prettytable-devel?hl=en.
>


--
Luke Maurits <lu...@maurits.id.au>
Reply all
Reply to author
Forward
0 new messages