Issue Details (XML | Word | Printable)

Key: MPXDOC-111
Type: Bug Bug
Status: Open Open
Priority: Major Major
Assignee: Unassigned
Reporter: Carlos Sanchez
Votes: 3
Watchers: 3
Operations

If you were logged in you would be able to see more operations.
Maven 1.x XDoc Plugin

<br/> is transformed into <br></br>

Created: 27/Jun/04 01:49 PM   Updated: 03/Aug/06 09:41 AM
Return to search
Component/s: None
Affects Version/s: None
Fix Version/s: None

Time Tracking:
Not Specified


 Description  « Hide

This causes two line breaks (at least in IE 6)

For example (at http://cvs.apache.org/viewcvs.cgi/maven-plugins/aspectj/xdocs/index.xml?rev=1.7&view=auto)

<section name="Installing">
<p>
To install or update the plugin do the following:<br/>
<code>maven plugin:download -DgroupId=maven -DartifactId=maven-aspectj-plugin -Dversion=<version></code>
</p>
</section>

is transformed into (at http://maven.apache.org/reference/plugins/aspectj/)

<div class="section">
<a name="Installing"></a><h2>Installing</h2>
<p>
To install or update the plugin do the following:<br></br>
<code>maven plugin:download -DgroupId=maven -DartifactId=maven-aspectj-plugin -Dversion=<version></code>
</p>
</div>



Arnaud Heritier made changes - 16/Aug/04 06:00 PM
Field Original Value New Value
Assignee Jason van Zyl [ jason ] Arnaud HERITIER [ aheritier ]
Arnaud Heritier made changes - 22/Aug/05 06:41 AM
Assignee Arnaud Heritier [ aheritier ]
Lukas Theussl added a comment - 30/Aug/05 05:36 PM

I don't see that in Mozilla nor in Konqueror so I assume it's an IE issue. In any case, <br/> and <br></br> are equivalent. Maybe somebody using IE can confirm that so we can close this issue.


Carlos Sanchez added a comment - 30/Aug/05 06:40 PM

It's an IE issue only, I don't know how difficult can be to generate collapsed tags when content it's empty


fabrizio giustina added a comment - 31/Aug/05 12:51 AM

the "two line break" visualization it's surely an IE issue but note that <br/> and <br></br> are not equivalent, br must be collapsed in html.


Brett Porter added a comment - 31/Aug/05 01:01 AM

I agree we should bew acommodating IE quirks where possible, but I don't want to invest effort in grappling with jsl at this point. Let's try and get to an m2-technology-based maven1 xdoc plugin as soon as possible...


Lukas Theussl added a comment - 31/Aug/05 05:41 PM

The expanding of empty elements is a dom4j issue, I have opened a bug report there. I don't think there is much to do here...


Arnaud Heritier added a comment - 03/Aug/06 05:59 AM

Isn't it fixed in the current plugin with the last upgrade of dom4J ? I'll check this...


Lukas Theussl added a comment - 03/Aug/06 08:55 AM

I checked it and it's still not fixed.

The bug I opened in dom4j is fixed in the version we are shipping with m11b3, but the feature has to be switched on explicitly, and it's an issue of jelly using dom4j, and I forgot to check that when I custom-built our own jelly, so it's not fixed yet...


Arnaud Heritier added a comment - 03/Aug/06 09:02 AM

Can we plan to upgrade our custom Jelly to fix this in the RC1 ?


Lukas Theussl added a comment - 03/Aug/06 09:22 AM

Another custom jelly build? If that carries on like that we might as well take over jelly maintenance... I don't think this issue is important enough for that...


Arnaud Heritier added a comment - 03/Aug/06 09:41 AM

One more, one less
But it's a good idea we can propose that you join the jakarta-commons team !!