WindowOverview: Dim background, keep other windows visible

Author: leolost2605

src/DesktopIntegration.vala

@@ -248,8 +248,12 @@ public class Gala.DesktopIntegration : GLib.Object {
             throw new IOError.NOT_FOUND ("App not found");
         }
 
+        var sorted_windows = wm.get_display ().sort_windows_by_stacking (app.get_windows ());
+
         uint64[] window_ids = {};
-        foreach (var window in app.get_windows ()) {
+        foreach (var window in sorted_windows) {
+            window.raise_and_make_recent_on_workspace (window.get_workspace ());
+            window.focus (Meta.CURRENT_TIME);
             window_ids += window.get_id ();
         }
 

src/Widgets/WindowOverview.vala

@@ -15,6 +15,7 @@ public class Gala.WindowOverview : ActorTarget, RootTarget, ActivatableComponent
 
     private GestureController gesture_controller; // Currently not used for actual touchpad gestures but only as controller
 
+    private Clutter.Actor background;
     private Clutter.Actor monitors;
     private ModalProxy modal_proxy;
 
@@ -32,6 +33,18 @@ public class Gala.WindowOverview : ActorTarget, RootTarget, ActivatableComponent
         };
         add_gesture_controller (gesture_controller);
 
+        background = new Clutter.Actor () {
+#if HAS_MUTTER47
+            background_color = Cogl.Color.from_string ("black")
+#else
+            background_color = Clutter.Color.from_string ("black")
+#endif
+        };
+        background.add_constraint (new Clutter.BindConstraint (this, SIZE, 0));
+        add_child (background);
+
+        add_target (new PropertyTarget (MULTITASKING_VIEW, background, "opacity", typeof (uint), 0u, 150u));
+
         monitors = new ActorTarget ();
         add_child (monitors);
     }
@@ -71,19 +84,17 @@ public class Gala.WindowOverview : ActorTarget, RootTarget, ActivatableComponent
         window_ids = hints != null && "windows" in hints ? (uint64[]) hints["windows"] : null;
 
         var windows = new List<Meta.Window> ();
+#if HAS_MUTTER48
+        foreach (unowned var window_actor in wm.get_display ().get_compositor ().get_window_actors ()) {
+#else
         foreach (unowned var window_actor in wm.get_display ().get_window_actors ()) {
+#endif
             var window = window_actor.meta_window;
-            if (ShellClientsManager.get_instance ().is_positioned_window (window)) {
-                continue;
-            }
-
-            if (window.window_type != Meta.WindowType.NORMAL &&
-                window.window_type != Meta.WindowType.DIALOG ||
+            if (ShellClientsManager.get_instance ().is_positioned_window (window) ||
+                window.window_type != NORMAL && window.window_type != DIALOG ||
                 window.is_attached_dialog () ||
                 window_ids != null && !(window.get_id () in window_ids)
             ) {
-                window_actor.hide ();
-
                 continue;
             }
 

src/WindowManager.vala

@@ -281,20 +281,20 @@ namespace Gala {
                 Meta.KeyBinding.set_custom_handler ("switch-group-backward", window_switcher.handle_switch_windows);
             }
 
-            if (plugin_manager.window_overview_provider == null
-                || (window_overview = (plugin_manager.get_plugin (plugin_manager.window_overview_provider) as ActivatableComponent)) == null
-            ) {
-                window_overview = new WindowOverview (this);
-                ui_group.add_child ((Clutter.Actor) window_overview);
-            }
-
             // Add the remaining components that should be on top
             shell_group = new Clutter.Actor ();
             ui_group.add_child (shell_group);
 
             menu_group = new Clutter.Actor ();
             ui_group.add_child (menu_group);
 
+            if (plugin_manager.window_overview_provider == null
+                || (window_overview = (plugin_manager.get_plugin (plugin_manager.window_overview_provider) as ActivatableComponent)) == null
+            ) {
+                window_overview = new WindowOverview (this);
+                ui_group.add_child ((Clutter.Actor) window_overview);
+            }
+
             var feedback_group = display.get_compositor ().get_feedback_group ();
             stage.remove_child (feedback_group);
             ui_group.add_child (feedback_group);