[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [cp-patches] Patch: BasicMenuItemUI fix
From: |
Lillian Angel |
Subject: |
Re: [cp-patches] Patch: BasicMenuItemUI fix |
Date: |
Fri, 21 Oct 2005 14:11:45 -0400 |
On Fri, 2005-10-21 at 17:31 +0200, Roman Kennke wrote:
> Hi Lillian,
>
> It looks a little strange to handle CheckBoxMenuItem specific cases in
> the superclass. However, I understand that the API of
> BasicCheckBoxMenuItem does not leave much room for specific handling of
> this.
I agree, I noticed this.
> What looks a bit strange to me is that the specs define a method
> processMouseEvent in BasicCheckBoxMenuItemUI and
> BasicRadioButtonMenuItemUI but _not_ in one of the superclasses. Could
> these method possibly have something to do with your problem? Can you
> make any sense of these methods? Who is supposed to call these? The
> MouseListener of the superclass? A private inner mouse listener? If it
> was a private mouse listener, where should it be set up? These classes
> don't override installListeners or something like that..
I will look into this
Thanks
Lillian