Hello,

This is the complete (merged) patch.

No documentation update is needed: This patch enables multi-row headers,
see test case below:

[cols="^,^,^",options="header"]
|=======================
.2+^.^|Supplier    2+^|Location
                      |Address         |City

|Exotic Liquids       |49 Gilbert St.  |London
|Cajun Delights       |P.O. Box 78934  |New Orleans
|=======================

Specifically, this patch will
1. make "Address" and "City" rendered with correct header style.
2. when the table is larger than one page, proper header content for
headers after (and including) the 2nd page.

Yung-ching


On Tue, Feb 2, 2016 at 10:52 AM, Lex Trotman <[email protected]> wrote:

> Sorry, I missed your previous patch.  Havn't had a chance to look at
> it in detail, but it needs the user guide updating to document how to
> set the number of header rows.
>
> Cheers
> Lex
>
> On 2 February 2016 at 11:58, Yung-ching Hsiao <[email protected]>
> wrote:
> > Hello,
> >
> > I just discovered that the previous patch is not complete: for multirow
> > headers, the 2nd header row is duplicated in both header and content
> rows.
> > It should have been removed from the rows[] array when it is recognized
> as
> > headers.
> >
> > The attached single line patch fixes this problem.
> >
> > Yung-ching
> >
> >
> > On Wednesday, September 23, 2015 at 3:58:05 PM UTC+8, Yung-ching Hsiao
> > wrote:
> >>
> >> Hi,
> >>
> >> We need this feature so I took a look and come up with a patch.
> >>
> >> I only modified the translate() member function in class
> >> Table(AbstractBlock). This change along seems to fix the problem for us.
> >> Could you take a look? My modification checks the row span of the first
> row
> >> and include that many rows as header rows.
> >>
> >> There is another translate() in class Tables(AbstractBlocks) that split
> >> header rows using a different mechanism. As it does not seem to be
> related
> >> to options="header" handling and I don't understand the purpose of that
> >> function, I'm not touching it.
> >>
> >> Yung-ching
> >>
> >> On Wednesday, February 18, 2015 at 11:32:23 AM UTC+8, Lex Trotman wrote:
> >>>
> >>> On 18 February 2015 at 03:58, russurquhart1 <[email protected]>
> >>> wrote:
> >>> > Hi,
> >>> >
> >>> > I understood, and have applied the header style to the cell. But
> when i
> >>> > generate the DocBook output, those cells with the header style are
> not
> >>> > enclosed with a thead element, i.e. there is not a header row.
> >>> >
> >>> > Is there a way to get the header formatting for this, and also get
> >>> > those row
> >>> > in the thead element?
> >>>
> >>> I don't think so.  The Asciidoc syntax has no way of specifying how
> >>> many rows to include in the thead element, so it includes one only.
> >>> This http://asciidoc.org/userguide.html#X74 does explicitly say "the
> >>> first row".
> >>>
> >>> So the only way of handling rowspans in the header is to manually
> >>> style them like the header without including a thead element.
> >>>
> >>> Multi-row headers would be a nice addition, patches or pull requests
> >>> welcome :)
> >>>
> >>> Cheers
> >>> Lex
> >>>
> >>> >
> >>> > On Tuesday, February 17, 2015 at 10:24:03 AM UTC-6, Lex Trotman
> wrote:
> >>> >>
> >>> >> On 18 February 2015 at 02:58, russurquhart1 <[email protected]>
> >>> >> wrote:
> >>> >> > Hi,
> >>> >> >
> >>> >> > This works in part but i think there may still be an issue.
> >>> >> >
> >>> >> > When i have the following:
> >>> >> >
> >>> >> > .Interrupt Vectors and Priorities
> >>> >> > [width="100%"]
> >>> >> > |====================
> >>> >> > .2+.>|Vector 2+^|+++<u>Absolute Address (hexadecimal)</u>+++
> >>> >> > .2+.>|Hardware
> >>> >> > Priority .2+.>|Description
> >>> >> > ^|VMAP = 0 ^|VMAP = 1
> >>> >> > |RESET |00{nbsp}0000 |3F{nbsp}FFCO | 1 (highest) | Reset
> >>> >> >
> >>> >> > The cells in the top two rows, which i would like to have
> contained
> >>> >> > in
> >>> >> > my
> >>> >> > header row, are aligned properly, with the VMAP = 0 cell and the
> >>> >> > VMAP =
> >>> >> > 1
> >>> >> > underneath the Absolute Address (hexidecimal) cell. But the row is
> >>> >> > not
> >>> >> > bold/strong.
> >>> >> >
> >>> >> > When i do the following, as you suggested:
> >>> >> >
> >>> >> > .Interrupt Vectors and Priorities
> >>> >> > [width="100%",options="header"]
> >>> >> > |====================
> >>> >> > .2+.>|Vector 2+^|+++<u>Absolute Address (hexadecimal)</u>+++
> >>> >> > .2+.>|Hardware
> >>> >> > Priority .2+.>|Description
> >>> >> > ^|VMAP = 0 ^|VMAP = 1
> >>> >> > |RESET |00{nbsp}0000 |3F{nbsp}FFCO | 1 (highest) | Reset
> >>> >> >
> >>> >> > The frist row, is bolded as i would expect a header to be, but the
> >>> >> > VMAP
> >>> >> > = 0
> >>> >> > and VMAP = 1 cells are on the next row, and are not contained
> within
> >>> >> > the
> >>> >> > header row. I checked the docbook xml that was generated, and this
> >>> >> > is
> >>> >> > true.
> >>> >> >
> >>> >> > So i guess how i'm trying to do the header row isn't right? Can
> you
> >>> >> > suggest
> >>> >> > a way to do this type of formatting?
> >>> >>
> >>> >> As I said, apply the header style to the cells, not to the table.
> See
> >>> >> http://asciidoc.org/userguide.html#X84
> >>> >>
> >>> >> Cheers
> >>> >> Lex
> >>> >>
> >>> >> >
> >>> >> > Thanks,
> >>> >> >
> >>> >> >
> >>> >> > Russ
> >>> >> >
> >>> >> > On Monday, February 16, 2015 at 5:10:06 PM UTC-6, Lex Trotman
> wrote:
> >>> >> >>
> >>> >> >> On 17 February 2015 at 10:00, russurquhart1 <
> [email protected]>
> >>> >> >> wrote:
> >>> >> >> > Hi,
> >>> >> >> >
> >>> >> >> > very often in the tables we create, we have cells in the header
> >>> >> >> > that
> >>> >> >> > span
> >>> >> >> > columns and rows. It seems i cant do this in asciidocfx, and
> the
> >>> >> >> > creatof
> >>> >> >> > asciifocfx, suggested i ask you guys..
> >>> >> >> >
> >>> >> >> > When i have something like this, it looks ok:
> >>> >> >> >
> >>> >> >> > .Interrupt Vectors and Priorities
> >>> >> >> > [width="100%"]
> >>> >> >> > |====================
> >>> >> >> > .2+.>s|Vector 2+^s|Absolute Address (hexadecimal)
> .2+.>s|Hardware
> >>> >> >> > Priority
> >>> >> >> > .2+.>s|Description
> >>> >> >> > ^s|VMAP = 0 ^s|VMAP = 1
> >>> >> >> > |RESET |00{nbsp}0000 |3F{nbsp}FFCO | 1 (highest) | Reset
> >>> >> >> > |INT1 |00 0002 |3F FFC2 |5 |Maskable interrupt 1
> >>> >> >> >
> >>> >> >> > But if i want the first row to be a header and i do, the
> >>> >> >> > following,
> >>> >> >> > it
> >>> >> >> > doesn't want to do it:
> >>> >> >> >
> >>> >> >> > .Interrupt Vectors and Priorities
> >>> >> >> > [width="100%",options="header"]
> >>> >> >> > |====================
> >>> >> >> > .2+.>s|Vector 2+^s|Absolute Address (hexadecimal)
> .2+.>s|Hardware
> >>> >> >> > Priority
> >>> >> >> > .2+.>s|Description
> >>> >> >> > ^s|VMAP = 0 ^s|VMAP = 1
> >>> >> >> > |RESET |00{nbsp}0000 |3F{nbsp}FFCO | 1 (highest) | Reset
> >>> >> >> > |INT1 |00 0002 |3F FFC2 |5 |Maskable interrupt 1
> >>> >> >> >
> >>> >> >>
> >>> >> >> You are now trying to apply two styles to the header cells, the
> >>> >> >> "header" style in the table attribute, and the "strong" style on
> >>> >> >> the
> >>> >> >> cell.  The cell style has priority so the "header" is ignored.
> You
> >>> >> >> can just apply the "header" style to the cells instead and it
> will
> >>> >> >> apply irrespective of their spans.
> >>> >> >>
> >>> >> >> Cheers
> >>> >> >> Lex
> >>> >> >>
> >>> >> >> >
> >>> >> >> > Is this something with how asciidoc works?
> >>> >> >> >
> >>> >> >> >
> >>> >> >> > Thanks,
> >>> >> >> >
> >>> >> >> >
> >>> >> >> > Russ
> >>> >> >> >
> >>> >> >> > --
> >>> >> >> > You received this message because you are subscribed to the
> >>> >> >> > Google
> >>> >> >> > Groups
> >>> >> >> > "asciidoc" group.
> >>> >> >> > To unsubscribe from this group and stop receiving emails from
> it,
> >>> >> >> > send
> >>> >> >> > an
> >>> >> >> > email to [email protected].
> >>> >> >> > To post to this group, send email to [email protected].
> >>> >> >> > Visit this group at http://groups.google.com/group/asciidoc.
> >>> >> >> > For more options, visit https://groups.google.com/d/optout.
> >>> >> >
> >>> >> > --
> >>> >> > You received this message because you are subscribed to the Google
> >>> >> > Groups
> >>> >> > "asciidoc" group.
> >>> >> > To unsubscribe from this group and stop receiving emails from it,
> >>> >> > send
> >>> >> > an
> >>> >> > email to [email protected].
> >>> >> > To post to this group, send email to [email protected].
> >>> >> > Visit this group at http://groups.google.com/group/asciidoc.
> >>> >> > For more options, visit https://groups.google.com/d/optout.
> >>> >
> >>> > --
> >>> > You received this message because you are subscribed to the Google
> >>> > Groups
> >>> > "asciidoc" group.
> >>> > To unsubscribe from this group and stop receiving emails from it,
> send
> >>> > an
> >>> > email to [email protected].
> >>> > To post to this group, send email to [email protected].
> >>> > Visit this group at http://groups.google.com/group/asciidoc.
> >>> > For more options, visit https://groups.google.com/d/optout.
> >
> > --
> > You received this message because you are subscribed to the Google Groups
> > "asciidoc" group.
> > To unsubscribe from this group and stop receiving emails from it, send an
> > email to [email protected].
> > To post to this group, send email to [email protected].
> > Visit this group at https://groups.google.com/group/asciidoc.
> >
> > For more options, visit https://groups.google.com/d/optout.
>
> --
> You received this message because you are subscribed to a topic in the
> Google Groups "asciidoc" group.
> To unsubscribe from this topic, visit
> https://groups.google.com/d/topic/asciidoc/SXYRiIysFhQ/unsubscribe.
> To unsubscribe from this group and all its topics, send an email to
> [email protected].
> To post to this group, send email to [email protected].
> Visit this group at https://groups.google.com/group/asciidoc.
> For more options, visit https://groups.google.com/d/optout.
>

-- 
You received this message because you are subscribed to the Google Groups 
"asciidoc" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/asciidoc.
For more options, visit https://groups.google.com/d/optout.

Attachment: multirow_header20160203.patch
Description: Binary data

Reply via email to