[PATCH] Keep the current spoke on top of the hub.

[Chris Lumens]

This means you can't alt-tab away from the spoke and do stuff on the hub
and really confuse anaconda.
---
 pyanaconda/ui/gui/TODO             | 2 --
 pyanaconda/ui/gui/hubs/__init__.py | 2 ++
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/pyanaconda/ui/gui/TODO b/pyanaconda/ui/gui/TODO
index 6f8dcf9..cb86ca4 100644
--- a/pyanaconda/ui/gui/TODO
+++ b/pyanaconda/ui/gui/TODO
@@ -13,8 +13,6 @@ Fedora 18
   where you go next.
 - Something causes the up and down arrows to resize the anaconda window.
 - Wall of anaconda needs to be taken into account.
-- When you display a spoke, the hub needs to be hidden so you can't alt-tab.  This is especially
-  bad on liveinst.
 - Why does the first hub jump left a little bit the first time you use the keyboard to navigate?
 - In custom storage, it'd be nice to be able to say something like "Use this
   previous installation as the starting storage configuration for the system
diff --git a/pyanaconda/ui/gui/hubs/__init__.py b/pyanaconda/ui/gui/hubs/__init__.py
index f5e211f..4d8c6e3 100644
--- a/pyanaconda/ui/gui/hubs/__init__.py
+++ b/pyanaconda/ui/gui/hubs/__init__.py
@@ -94,6 +94,7 @@ class Hub(GUIObject, common.Hub):
         action.window.set_beta(self.window.get_beta())
         action.window.set_property("distribution", self.window.get_property("distribution"))

+        action.window.set_transient_for(self.window)
         action.window.show_all()

# Start a recursive main loop for this spoke, which will prevent
@@ -101,6 +102,7 @@ class Hub(GUIObject, common.Hub):
         # prevent the user from switching away.  It's up to the spoke's back
         # button handler to kill its own layer of main loop.
         Gtk.main()
+        action.window.set_transient_for(None)

if not action.skipTo or (action.skipTo and action.applyOnSkip):
             action.apply()

[Vratislav Podzimek]

Ack.

On Thu, 2012-08-30 at 16:39 -0400, Chris Lumens wrote:
> This means you can't alt-tab away from the spoke and do stuff on the hub
> and really confuse anaconda.
>
> pyanaconda/ui/gui/TODO             | 2 --
>  pyanaconda/ui/gui/hubs/__init__.py | 2 ++
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/pyanaconda/ui/gui/TODO b/pyanaconda/ui/gui/TODO
> index 6f8dcf9..cb86ca4 100644
> --- a/pyanaconda/ui/gui/TODO
> +++ b/pyanaconda/ui/gui/TODO
> @@ -13,8 +13,6 @@ Fedora 18
>    where you go next.
> Something causes the up and down arrows to resize the anaconda window.
> Wall of anaconda needs to be taken into account.
>
> -- When you display a spoke, the hub needs to be hidden so you can't alt-tab.  This is especially
> bad on liveinst.
> Why does the first hub jump left a little bit the first time you use the keyboard to navigate?
> In custom storage, it'd be nice to be able to say something like "Use this
> previous installation as the starting storage configuration for the system
>
> diff --git a/pyanaconda/ui/gui/hubs/__init__.py b/pyanaconda/ui/gui/hubs/__init__.py
> index f5e211f..4d8c6e3 100644
> --- a/pyanaconda/ui/gui/hubs/__init__.py
> +++ b/pyanaconda/ui/gui/hubs/__init__.py
> @@ -94,6 +94,7 @@ class Hub(GUIObject, common.Hub):
>          action.window.set_beta(self.window.get_beta())
>          action.window.set_property("distribution", self.window.get_property("distribution"))
>    action.window.set_transient_for(self.window)
>    action.window.show_all()
>
>    # Start a recursive main loop for this spoke, which will prevent
>
> @@ -101,6 +102,7 @@ class Hub(GUIObject, common.Hub):
>          # prevent the user from switching away.  It's up to the spoke's back
>          # button handler to kill its own layer of main loop.
>          Gtk.main()
>    action.window.set_transient_for(None)
>
>    if not action.skipTo or (action.skipTo and action.applyOnSkip):
>        action.apply()