391103 - Invalid HTML on Firefox page (due to eBay-edition block)

Status: RESOLVED FIXED

(www.mozilla.org :: General, defect)

Description

[Pascal Chevrel:pascalc (PTO until April 26)]

http://www.mozilla.com/en-US/firefox/ is no longer XHTML vadid because the "special edition" box is not correct HTML (inline level element <a> including a block-level <p> element).

http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Fwww.mozilla.com%2Fen-US%2Ffirefox%2F

I have corrected the HTML on mozilla-europe (http://www.mozilla-europe.org/en/products/firefox/), here is the valid version:

<div id="specialeditions">
<h3>Special Editions</h3>
<p>
<a href="http://en-gb.www.mozilla.com/en-GB/add-ons/ebay/">
<img src="/img/firefox/feature-ebay-logo.png" alt="ebay logo" />
<span class="specialedition-title">Mozilla Firefox - eBay Edition</span>Be a better bidder. Download now.
</a>
</p>
</div> <!-- end #specialeditions -->

one CSS change :

#specialeditions p { display: block; margin: 0; }

Comment 1

[Steven Garrity [:sgarrity]]

Fixed in r5569. Tagged for production in r5572. Reed, can you push live?

Thanks for the code Pascal.

Comment 2

[Paul Kim]

i'll file a bug to sync mozilla.com.