[prev in list] [next in list] [prev in thread] [next in thread]
List: kfm-devel
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 " " 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  moretext but does allow   moretext (or of course the
> prope 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]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic