Re: [cp-patches] CheckboxMenuItem patch

[Craig Black]

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.

Craig


2005-02-10  Craig Black  <address@hidden>

  * gnu/java/awt/peer/gtk/GtkCheckboxMenuItem.java
  (postMenuActionEvent): Implement to notify ItemListeners.
  * java/awt/CheckboxMenuItem.java
  (dispatchEventImpl): Update state on ItemEvent.
  * java/awt/MenuItem.java
  (processActionEvent): Retarget event source.

checkboxmenuitem.patch
Description: Text document