[ 
http://jira.codehaus.org/browse/DOXIA-177?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lukas Theussl reopened DOXIA-177:
---------------------------------


Vincent, I just noticed that this patch creates a backward compat problem 
because it is not possible anymore to create nested tables.
Eg the following snippet from generateDependencyDetails method in PIR's 
DependencyConvergenceReport causes an Exception:

{code}
        sink.table();
        sink.tableRow();
        ...
        sink.tableCell();
        sink.table();
        ...
        sink.table_();
        sink.tableCell_();
        sink.tableRow_();
        sink.table_();
{code}

because the tableWriter is closed in the first table_() call. I think this is 
more severe than the original problem of invalid xhtml, so if it's not easily 
fixed, I'd prefer to revert this.

> Invalid XHTML because of wrong position of table caption
> --------------------------------------------------------
>
>                 Key: DOXIA-177
>                 URL: http://jira.codehaus.org/browse/DOXIA-177
>             Project: Maven Doxia
>          Issue Type: Bug
>          Components: Module - Xhtml
>    Affects Versions: 1.0-alpha-9
>            Reporter: Lukas Theussl
>            Assignee: Vincent Siveton
>             Fix For: 1.1
>
>         Attachments: DOXIA-177.diff
>
>
> Currently a table caption is most of the time emitted after the table body 
> (mainly because apt puts the caption after the table). However, for a valid 
> xhtml-1.0, the table caption has to come before the first table row.
> Since we don't have a mechanism yet to enforce the order of parsing events 
> (see DOXIA-132), and I am not sure if this can be achieved in general, any 
> sink should be flexible enough to deal with any reasonable order of events, 
> to produce some valid output. 

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to