Re: svn commit: r580725 - /maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java

View: New views
3 Messages — Rating Filter:   Alert me  

Parent Message unknown Re: svn commit: r580725 - /maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java

by Vincent Siveton :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi,

This fix is an important issue for report generation. I discussed on irc with
Dennis about that and he will deploy new snapshots to test again MPIR.
If everything will be ok, we think we need to release an alpha-10 of
doxia. What others think?

Cheers,

Vincent

2007/9/30, vsiveton@... <vsiveton@...>:

> Author: vsiveton
> Date: Sun Sep 30 06:37:06 2007
> New Revision: 580725
>
> URL: http://svn.apache.org/viewvc?rev=580725&view=rev
> Log:
> o fixed typo
>
> Modified:
>     maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java
>
> Modified: maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java
> URL: http://svn.apache.org/viewvc/maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java?rev=580725&r1=580724&r2=580725&view=diff
> ==============================================================================
> --- maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java (original)
> +++ maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java Sun Sep 30 06:37:06 2007
> @@ -861,7 +861,7 @@
>
>          if ( width != null )
>          {
> -            att.addAttribute( Attribute.COLSPAN, width );
> +            att.addAttribute( Attribute.WIDTH, width );
>          }
>
>          if ( justif != null )
>
>
>

Re: svn commit: r580725 - /maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java

by Dennis Lundberg-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

New are SNAPSHOTs deployed now.

Vincent Siveton wrote:

> Hi,
>
> This fix is an important issue for report generation. I discussed on irc with
> Dennis about that and he will deploy new snapshots to test again MPIR.
> If everything will be ok, we think we need to release an alpha-10 of
> doxia. What others think?
>
> Cheers,
>
> Vincent
>
> 2007/9/30, vsiveton@... <vsiveton@...>:
>> Author: vsiveton
>> Date: Sun Sep 30 06:37:06 2007
>> New Revision: 580725
>>
>> URL: http://svn.apache.org/viewvc?rev=580725&view=rev
>> Log:
>> o fixed typo
>>
>> Modified:
>>     maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java
>>
>> Modified: maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java
>> URL: http://svn.apache.org/viewvc/maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java?rev=580725&r1=580724&r2=580725&view=diff
>> ==============================================================================
>> --- maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java (original)
>> +++ maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java Sun Sep 30 06:37:06 2007
>> @@ -861,7 +861,7 @@
>>
>>          if ( width != null )
>>          {
>> -            att.addAttribute( Attribute.COLSPAN, width );
>> +            att.addAttribute( Attribute.WIDTH, width );
>>          }
>>
>>          if ( justif != null )
>>
>>
>>
>


--
Dennis Lundberg

Re: svn commit: r580725 - /maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java

by Lukas Theussl-4 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

I have reverted the whole patch at DOXIA-78 (it also affected the
XdocParser), and also my erroneous javadoc comments in the Sink. If you
want to go for an alpha-10, you should merge those changes into the branch.

-Lukas


Vincent Siveton wrote:

> Hi,
>
> This fix is an important issue for report generation. I discussed on irc with
> Dennis about that and he will deploy new snapshots to test again MPIR.
> If everything will be ok, we think we need to release an alpha-10 of
> doxia. What others think?
>
> Cheers,
>
> Vincent
>
> 2007/9/30, vsiveton@... <vsiveton@...>:
>
>>Author: vsiveton
>>Date: Sun Sep 30 06:37:06 2007
>>New Revision: 580725
>>
>>URL: http://svn.apache.org/viewvc?rev=580725&view=rev
>>Log:
>>o fixed typo
>>
>>Modified:
>>    maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java
>>
>>Modified: maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java
>>URL: http://svn.apache.org/viewvc/maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java?rev=580725&r1=580724&r2=580725&view=diff
>>==============================================================================
>>--- maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java (original)
>>+++ maven/doxia/doxia/trunk/doxia-modules/doxia-module-xhtml/src/main/java/org/apache/maven/doxia/module/xhtml/XhtmlSink.java Sun Sep 30 06:37:06 2007
>>@@ -861,7 +861,7 @@
>>
>>         if ( width != null )
>>         {
>>-            att.addAttribute( Attribute.COLSPAN, width );
>>+            att.addAttribute( Attribute.WIDTH, width );
>>         }
>>
>>         if ( justif != null )
>>
>>
>>