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

List:       kfm-devel
Subject:    Re: [PATCH] #69559
From:       David Faure <dfaure () klaralvdalens-datakonsult ! se>
Date:       2003-12-08 19:32:41
[Download RAW message or body]

On Monday 08 December 2003 20:08, Lubos Lunak wrote:
> On Monday 08 of December 2003 18:10, Koos Vriezen wrote:
> > On Mon, Dec 08, 2003 at 06:02:55PM +0100, Lubos Lunak wrote:
> > > On Monday 08 of December 2003 17:50, Koos Vriezen wrote:
> > > > On Fri, Dec 05, 2003 at 03:17:42PM +0100, Lubos Lunak wrote:
> > > > >  Oh, and this time I also didn't forget the reverse mapping in
> > > > > RenderWidget ;).
> > > >
> > > > But frames are forgotten again, no?
> > > > (I use http://java.sun.com/j2se/1.4.2/docs/api/index.html when working
> > > >  on kjas)
> > >
> > >  I thought that has been already fixed by David? Do you have any problems
> > > with them?
> >
> > They don't scroll with the scroll keys.
> >
> > > > Koos
> 
>  I've found the reason, it was this David's change 
> http://lists.kde.org/?l=kde-cvs&m=107064986608639&w=2 . Specifically, the 
> attached patch fixes your problem. However, I have absolutely no idea what 
> that code does, so the patch may be wrong *shrug*. I wonder why the 
> dispatching is done there, as if the event is not filtered out here, the 
> dispatching will be done once again in KHTMLView, but then I have no clue 
> about this.

Hmm, indeed. I wonder if this code isn't simply a leftover from the old way
of doing things (i.e. before Lars' changes). Could someone test removing
the whole "case:" in this switch?

-- 
David Faure -- faure@kde.org, dfaure@klaralvdalens-datakonsult.se
Qt/KDE/KOffice developer
Klarälvdalens Datakonsult AB, Platform-independent software solutions

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