Maven 1.x XDoc Plugin

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

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

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>

Activity

Hide
Lukas Theussl added a comment -

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.

Show
Lukas Theussl added a comment - 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.
Hide
Carlos Sanchez added a comment -

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

Show
Carlos Sanchez added a comment - It's an IE issue only, I don't know how difficult can be to generate collapsed tags when content it's empty
Hide
fabrizio giustina added a comment -

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.

Show
fabrizio giustina added a comment - 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.
Hide
Brett Porter added a comment -

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...

Show
Brett Porter added a comment - 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...
Hide
Lukas Theussl added a comment -

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...

Show
Lukas Theussl added a comment - 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...
Hide
Arnaud Heritier added a comment -

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

Show
Arnaud Heritier added a comment - Isn't it fixed in the current plugin with the last upgrade of dom4J ? I'll check this...
Hide
Lukas Theussl added a comment -

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...

Show
Lukas Theussl added a comment - 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...
Hide
Arnaud Heritier added a comment -

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

Show
Arnaud Heritier added a comment - Can we plan to upgrade our custom Jelly to fix this in the RC1 ?
Hide
Lukas Theussl added a comment -

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...

Show
Lukas Theussl added a comment - 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...
Hide
Arnaud Heritier added a comment -

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

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

People

Vote (3)
Watch (3)

Dates

  • Created:
    Updated: