Re: [cp-patches] Patch: BasicMenuItemUI fix

[Lillian Angel]

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