pivot-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sandro Martini <sandro.mart...@gmail.com>
Subject Re: Disappearing Palettes
Date Tue, 24 Apr 2012 13:46:34 GMT
Ok, this need more time to better investigate so I didn't commit this,
and let's update later ...

Sandro


2012/4/2 Sandro Martini <sandro.martini@gmail.com>:
> Hi all,
> some week ago Piotr discovered a little issue related on Palette
> show/hide (Palettes check if their owner is active or if their owner
> contains the active window), and sent me a candidate patch ...
>
> This is a little updated version of it (with the old behaviour as last option):
>
> ### Eclipse Workspace Patch 1.0
> #P wtk
> Index: src/org/apache/pivot/wtk/Window.java
> ===================================================================
> --- src/org/apache/pivot/wtk/Window.java        (revision 1307386)
> +++ src/org/apache/pivot/wtk/Window.java        (working copy)
> @@ -955,7 +955,11 @@
>      * Clears the active window.
>      */
>     public static void clearActive() {
> -        setActiveWindow(null);
> +       Window owner = null;
> +        if (activeWindow != null && !activeWindow.isBottomMost())
> +               owner = activeWindow.getOwner();
> +
> +        setActiveWindow(owner);
>     }
>
>     /**
>
> note that still it's not committed (because I've never seen that
> problem) ... but even after applying it, all works good even in my
> tests (with WindowTest.java).
>
> Unless objections I'll commit in trunk by mid of this week.
>
> Bye,
> Sandro

Mime
View raw message