[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


reply via email to

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