225938 - Template version comment outputted before doctype

Status: RESOLVED DUPLICATE of bug 228894

(Bugzilla :: Bugzilla-General, defect)

Description

[Tama Pugsley]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5) Gecko/20031007 Firebird/0.7
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5) Gecko/20031007 Firebird/0.7

I've been customising the bugzilla interface to make it Project Manager
friendly. At the top of every template is the following comment:

<!-- [e-mail] -->

Which means Bugzilla is outputting page content before the doctype is written in
header.html.tmpl. I'm not sure if this breaks the XHTML DTD (I haven't looked)
but IE certainly doesn't like it and reverts to quirks mode. I've had to go
through and remove all the <!-- [e-mail] --> comments so the css only
pages will render properly. Maybe these comments could be moved to be template
comments i.e.

[%# [e-mail] %]

Reproducible: Always

Steps to Reproduce:
1.
2.
3.
Actual Results:  
n/a

Comment 1

[Dave Miller [:justdave]]

This bug is a duplicated of a WONTFIX (which I can't find right now) because
yes, that is legal in the DTD, as long as there's nothing other than comments
prior to the doctype declaration.

However, the allegation that it breaks Internet Explorer is new information, and
unfortunately might be reason enough to change it even if it's legal :(

Comment 2

[Ludovic Dubost]

I confirm that it breaks IE6... I'm trying to make the templates CSS friendly
and having this in the templates forbids to switch to strict mode and have IE6
use the same CSS box model.

It is something we should fix in order to allow to create interfaces using CSS
that are standard and understood by at least IE6 in the Internet Explorer family..

How about removing these comments and only keep one in the header template after
the DOCTYPE.. 

It also bloats the HTML code.. 

Comment 3

[Bradley Baetz (:bbaetz)]

We haven't been updating these versions, and I still claim that 2.16->2.18
involves changing almost every template anyway.....

Comment 4

[GavinS]

Resolving as a dupe of the bug which changed all the comments from html to
Template-stylee

*** This bug has been marked as a duplicate of 228894 ***