[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice-devel
Subject:    Re: KPresenter's Webpresentation
From:       Lukáš Tinkl <lukas.tinkl () suse ! cz>
Date:       2002-12-11 9:18:41
[Download RAW message or body]

Dne st 11. prosince 2002 09:00 Nicolas Goutte napsal(a):
> Having looked into KPresenter's file webpresentation.cc , I have noticed a
> few subtle errors.
>
> The subtle errors that I have found are:
> - title can be chosen by user, but the encoding declaration meta tag is
> after the title, so in fact the title cannot have characters of the chosen
> encoding.
> - characters not in the chosen encoding are not escaped.
> - special XML characters ( <, >, &, ") are not escaped at all.
> - QString::arg is used with multiple user defined strings. If a user
> specifies for any reason a sequence number (for example %2) the result
> might not be what is expected.
>
> (I am also unsure if a <P> element is allowed in a <BLOCKQUOTE> element.)
>
> The solution is quite easy, as KWord's HTML export filter has had to go
> around those problems too. ;-) As for QString::arg, I plan to change
> everything to QTextStream, which avoids to create a potentially big
> QString.
>
> As I am going to fix them, I am asking if I can add a XHTML 1.0 modus to
> webpresentation. That would mean that the HTML tags would be in lower case
> instead of being currently in upper case. (HTML do not care, XHTML wants
> lower case tags only.)

I'd say go ahead, I pretty much agree on all the items

-- 
Lukáš Tinkl
KDE developer
---------------------------------------------------------------------
SuSE CR, s.r.o.  e-mail: lukas.tinkl@suse.cz, lukas@kde.org
Drahobejlova 27  tel: +420 2 9654 2373
190 00 Praha 9   fax: +420 2 9654 2374
Czech Republic   http://www.suse.cz/


_______________________________________________
koffice-devel mailing list
koffice-devel@mail.kde.org
http://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic