[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [cp-patches] CheckboxMenuItem patch
From: |
Thomas Fitzsimmons |
Subject: |
Re: [cp-patches] CheckboxMenuItem patch |
Date: |
Thu, 10 Feb 2005 20:04:24 -0500 |
On Thu, 2005-02-10 at 13:24 -0700, Craig Black wrote:
> On Tue, 8 Feb 2005, Thomas Fitzsimmons wrote:
>
> > On Wed, 2005-02-02 at 11:58 -0700, Craig Black wrote:
> > > Hello,
> > > The following patch makes CheckboxMenuItem notify ItemListeners on
> > > selection/deselection.
> > > See attached test, comments?
> > >
> >
> > The test still doesn't work perfectly. The checkbox action should be
> > detected in the menu labeled "Menu" but it's reported in a menu labeled
> > "Checkbox". Also, the selected/deselected state is always reported as
> > selected.
> >
> > Tom
> >
> >
>
> Sorry about that, I've attached a modified patch that fixes both problems.
>
Thanks! Please commit.
Tom