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

List:       kde-i18n
Subject:    Re: patch for #776 - khtmlw bug fixed
From:       David Faure <faure () kde ! org>
Date:       1999-03-21 19:47:49
[Download RAW message or body]

On Sun, Mar 21, 1999 at 11:12:16AM -0800, Alex Zepeda wrote:
> On Sun, 21 Mar 1999, David Faure wrote:
> 
> > > >I added the check for the trailing ';' that one can expect in an entity.
> > > >The ';' was assumed, before, hence the problem with such URLs as the ones in the
> > > >bug report, because &nu and &ge are known entities.
> > > >
> > > >Are there HTML pages that don't provide the trailing ';' for entities ? The 
> > > >patch would break the displaying of such entities, but as it's bad HTML anyway...
> > 
> > > there are some pages not using the trailing ';', so it'll probably "break some
> > > broken pages"..., but I think we should apply it anyway. 
> > > First of all, it's better to have one char entity displayed wrongly than a
> > > link, that doesn't work, and second of all, if web page authors are using
> > > broken html, it's moslty their own fault, if the page is not displayed
> > > correctly. We can't account for every bad habit people have, when composing web
> > > pages...
> > 
> > Hi Lars.
> > 
> > I agree with you, but I'm afraid of the number of pages this would break.
> > http://www.ypeu.net, for instance, (after selecting a language)
> > displays "&nbsp" all over the pages.
> > 
> > (I wrote to that site, BTW.)
> > 
> > I'm afraid more pages use broken entities than links containing a query part...
> 
> Lynx already is broken on them, and I think Navigator does break when you
> have &nbspmoretext but does allow &nbsp moretext (or of course the
> prope &nbsp;moretext...

Hmm, that would be a good comprise I think : checking that the entity 
ends with ';' OR ' '.
The URLs having a query part can't include spaces, can they ? (they are
encoded afaik).

-- 
David FAURE
david.faure@insa-lyon.fr, faure@kde.org
http://www.insa-lyon.fr/People/AEDI/dfaure/index.html 
KDE, Making The Future of Computing Available Today

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