Merge lp:~madsa/slingshot/fix-app-focus into lp:~elementary-pantheon/slingshot/trunk

[Matt Spaulding]

[Cody Garver (codygarver)]

This doesn't fix the problem for me, I used Calculator as a test and the behavior is unchanged. It starts focused, then goes unfocused.

[Matt Spaulding (madsa)]

I didn't test with Calculator, but for me when I run Calculator it is focused with or without the patch. I don't seem to have a problem with that one. And I'll also note that the loss of focus only happens when another window is already open.

The two apps I really see a problem with are Terminal and AppCenter. The other apps I've tried now don't seem to have the focus problem. You might try testing with those two.

[Zisu Andrei (matzipan)]

Rejecting since it did not fix the issue for Cody.

Unmerged revisions

696. By Matt Spaulding on 2016-10-02

Fix app focus when opening from search

=== modified file 'lib/synapse-core/common-actions.vala'
--- lib/synapse-core/common-actions.vala	2014-06-04 19:35:44 +0000
+++ lib/synapse-core/common-actions.vala	2016-10-02 05:17:22 +0000
@@ -93,8 +93,7 @@
 
           try
           {
-            var display = Gdk.Display.get_default ();
-            app.launch (null, display.get_app_launch_context ());
+            app.launch (null, null);
             
             RelevancyService.get_default ().application_launched (app);
           }
@@ -152,8 +151,7 @@
             AppInfo app = AppInfo.create_from_commandline (
               original.get_commandline (), original.get_name (),
               AppInfoCreateFlags.NEEDS_TERMINAL);
-            var display = Gdk.Display.get_default ();
-            app.launch (null, display.get_app_launch_context ());
+            app.launch (null, null);
           }
           catch (Error err)
           {