[Kexi-devel] htmlmigrate.cpp

[Sebastian Sauer]

On Saturday 19 November 2005 19:40, Adam Pigg wrote:
> I think this implements all suggestions so far, keep them coming and i'll
> create a todo

- </TITLE>
+ </title>
cause XML/XHTML is case-sensitive.

- <meta http-equiv=\"Content-Type\" content=\"text/html; charset=UTF-8\">
+ <meta http-equiv=\"Content-Type\" content=\"text/html; charset=UTF-8\" />
cause we don't have a </meta> for it, so it's a standalone-tag.

maybe let's remove the
<div align=\"center\">
and it's matching </div> ? The div is for style only what could be moved to 
the style-definition with something like;
<style type="text/css">
[...]
table { text-align: center; }
</style>
That way XML-parsers are able to access the data at html => head => body => 
table rather then html => head => body => div (unnecessary) => table.

- <table border=0>
+ <table border="0">
cause in XML/XHTML attributes are always key="value" pairs with quotes.

m_migrateData->exportObjectName and QDate::currentDate().toString() should be 
HTML-escaped too imho, or?

-- 
Sebastian Sauer aka dipesh[sebsauer]
http://www.dipe.org/public_key.asc
Fingerprint: 8F1E 219B 16E6 4EC7 29CC F408 E193 65E2 9134 2221
Coder in http://www.koffice.org/kexi && http://www.kmldonkey.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/kexi-devel/attachments/20051120/09ad0248/attachment.pgp