classpath-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [cp-patches] [patch] fix GTK scrollbar peer


From: Thomas Fitzsimmons
Subject: Re: [cp-patches] [patch] fix GTK scrollbar peer
Date: Wed, 25 May 2005 17:01:23 -0400

On Wed, 2005-05-25 at 09:35 +0200, Michael Koch wrote:
> On Tue, May 24, 2005 at 09:18:38PM -0400, Thomas Fitzsimmons wrote:
> > Hi,
> > 
> > This patch fixes a long-standing bug in the GTK scrollbar peer.  The
> > proper fix required some GTK work:
> > 
> > http://bugzilla.gnome.org/show_bug.cgi?id=133263
> > 
> > The patch there landed in GTK 2.6.  The attached patch, which I
> > committed to GNU Classpath HEAD, changes the configure check to require
> > GTK 2.6 and takes advantage of the new "change-value" signal.  This
> > fixes two aspects of scrolling:
> > 
> > - adjustment types are now supported properly (UNIT_INCREMENT/DECREMENT,
> > BLOCK_INCREMENT/DECREMENT and TRACK type adjustments).
> > 
> > - setting a scrollbar's value works, without causing a callback loop
> > (callback loops cause strange behaviour, where e.g. two scrollbars
> > linked together will start oscillating crazily forever as soon as one is
> > adjusted)
> > 
> > Tom
> > 
> > 2005-05-24  Thomas Fitzsimmons  <address@hidden>
> > 
> >     * native/jni/gtk-peer/gnu_java_awt_peer_gtk_GtkScrollBarPeer.c
> >     (connectSignals): Connect to new change-value signal.
> >     (post_change_event): Remove signal handler.
> >     (slider_moved_cb): New signal handler.
> >     * configure.ac: Require gtk+-2.0 >= 2.6.
> 
> Please announce such updates on address@hidden before commiting so people
> can update their GTK. Another thing you missed was updating the NEWS and 
> INSTALL
> files to tell the people what minimum requirements we have now.

Sorry about that; I updated NEWS and INSTALL.  I'll inform
address@hidden of dependency changes from now on.

Tom






reply via email to

[Prev in Thread] Current Thread [Next in Thread]