On Sun, May 20, 2007 at 07:08:59PM +0200, Peter Kümmel wrote:
> Abdelrazak Younes wrote:

...
 
> OK, I've found a better solution ;)
> I had to test if it was worth to upgrade to 1GB ram.
> 
> > 
> > I can understand that. I was thinking of collecting the scroll events
> > and cancelling them if a scroll operation is still on-going but maybe
> > that's what your patch do. I only briefly looked at it and it _seems_
> > very complicated.
> 
> Yes this is what it does. An it _is_ complicated. ;)
> 
> But I've found a more Qt like and elegant solution.
> See attached patch, which should be much more readably.
> 
> I've introduced two events which are only posted once
> to the event queue until the event is not processed.
> 
> This also improves(?) the scrolling under Windows,
> see the comment in generateLyxScrollEvent.

Yes, this looks _much_ better. Just as complicated, but very well designed ;-/

- Martin
 

Reply via email to