Ticket #2379 (closed good practice: fixed)

Opened 21 months ago

Last modified 21 months ago

Useless XML prolog in the current templates

Reported by: nuxwin Owned by: nuxwin
Priority: normal Milestone: ispCP ω 1.0.7
Component: Frontend (GUI) Version: ispCP ω 1.0.5
Severity: Easy Keywords:
Cc:

Description (last modified by nuxwin) (diff)

The common recommendation is to not added the XML prolog in documents that are not treated/served like that. The XML prolog is only required for the XHTML 1.0 strict documents that is not your case.

For the XHTML documents, the reason to added the XML prolog is to resolve encoding problem with no ASCII characters but this problem don't exist with UTF8, UTF-16. Only the meta data for the charset should be present.

Change History

comment:1 Changed 21 months ago by nuxwin

  • Description modified (diff)

comment:2 Changed 21 months ago by nuxwin

  • Status changed from new to assigned

comment:3 Changed 21 months ago by nuxwin

  • Status changed from assigned to closed
  • Resolution set to fixed

comment:4 Changed 21 months ago by benedikt

I disagree with some of your changes. According to  HTML standards, each HTML document requires a document type declaration. Because most templates are no valid HTML so far, at least the XHTML 1.0 conform files should contain their DOCTYPE.

comment:5 Changed 21 months ago by nuxwin

I do not agree either. Either we put a Doctype in all documents (x)HTML or not at all! I know you likes the job half done, but it is not my case. We should be coherent and linear in our working!

comment:6 Changed 21 months ago by benedikt

But I don't understand why you removed it from the Error Docs and Default page, they have nothing to do with the rest of the admin panel. This just creates double work if we have to put it back. Has nothing to do with half done jobs.

comment:7 Changed 21 months ago by nuxwin

Ok, I'll revert back changes for these documents.

Note: See TracTickets for help on using tickets.