classpath-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [cp-patches] Patch for Approval fixes bug #16540 GlassPane intercept


From: Thomas Fitzsimmons
Subject: Re: [cp-patches] Patch for Approval fixes bug #16540 GlassPane intercepting Mouse Events
Date: Tue, 06 Sep 2005 16:40:06 -0400

On Tue, 2005-09-06 at 16:16 -0400, Anthony Balkissoon wrote:
> This patch (pending approval) fixes bug #16540 where a RootPaneContainer
> whose GlassPane was set to visible but had no MouseListeners attached
> was still intercepting Mouse Events.
> 
> The patch basically consists of the new implementation method
> getDeepestComponentForMouseEventAt which (like
> AWTUtilities.getDeepestComponentAt) simply calls findComponentAt.
> However, getDeepestComponentForMouseEventAt also checks to see if there
> is a visible GlassPane with no MouseListeners attached, in which case it
> calls findNonGlassPaneComponentAt which does the same thing as
> findComponentAt except that it ignores the GlassPane.
> 
> Note that I also added the API docs for findComponentAt.  Sorry to mix
> doc changes with code changes, but it was very similar to the docs I
> wrote for my new method findNonGlassPaneComponentAt.

The implementation seems fine but it should be moved into Swing.  I
don't think Container should need to know about Swing components.

Tom






reply via email to

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